Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TiFlash result is wrong #8904

Closed
lilinghai opened this issue Apr 7, 2024 · 1 comment · Fixed by #8927
Closed

TiFlash result is wrong #8904

lilinghai opened this issue Apr 7, 2024 · 1 comment · Fixed by #8927
Assignees
Labels

Comments

@lilinghai
Copy link

lilinghai commented Apr 7, 2024

Bug Report

Please answer these questions before submitting your issue. Thanks!

1. Minimal reproduce step (Required)

prepare some data
do random dml and check select count(*) using tiflash and tikv
upgrade from 6.17 to master, when one tiflash version is 6.1.7 and another is master, query from tiflash is wrong

mysql> begin;
Query OK, 0 rows affected (0.04 sec)

mysql> set tidb_isolation_read_engines='tiflash';
Query OK, 0 rows affected (0.03 sec)

mysql> select count(*) from t;
+----------+
| count(*) |
+----------+
| 15284754 |
+----------+
1 row in set (0.17 sec)

mysql> set tidb_isolation_read_engines='tikv';
Query OK, 0 rows affected (0.03 sec)

mysql> select count(*) from t;
+----------+
| count(*) |
+----------+
| 16117328 |
+----------+
1 row in set (0.15 sec)

mysql> commit;
Query OK, 0 rows affected (0.04 sec)

2. What did you expect to see? (Required)

3. What did you see instead (Required)

4. What is your TiFlash version? (Required)

@JaySon-Huang
Copy link
Contributor

JaySon-Huang commented Apr 9, 2024

Confirm it is introduced by #8854. Has been revert in the master/release-8.1 branch

@JaySon-Huang JaySon-Huang changed the title TiFlash result is wrong when upgrade TiFlash result is wrong Apr 9, 2024
ti-chi-bot bot pushed a commit that referenced this issue Apr 11, 2024
ti-chi-bot bot pushed a commit that referenced this issue Apr 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants