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

autocommit handles process issues #36581

Closed
keeplearning20221 opened this issue Jul 26, 2022 · 3 comments · Fixed by #36631
Closed

autocommit handles process issues #36581

keeplearning20221 opened this issue Jul 26, 2022 · 3 comments · Fixed by #36631
Assignees
Labels
affects-4.0 This bug affects 4.0.x versions. affects-5.0 This bug affects 5.0.x versions. affects-5.1 This bug affects 5.1.x versions. affects-5.2 This bug affects 5.2.x versions. affects-5.3 This bug affects 5.3.x versions. affects-5.4 This bug affects 5.4.x versions. affects-6.0 affects-6.1 affects-6.2 severity/major sig/transaction SIG:Transaction type/bug This issue is a bug.

Comments

@keeplearning20221
Copy link
Contributor

keeplearning20221 commented Jul 26, 2022

Bug Report

Please answer these questions before submitting your issue. Thanks!

1. Minimal reproduce step (Required)

set autocommit = ON;
begin;
Query OK, 0 rows affected (0.00 sec)

mysql> select * from test.t1;
+-----+
| id |
+-----+
| 590 |
+-----+
1 row in set (0.01 sec)

mysql> update test.t1 set id = id+1;
Query OK, 1 row affected (0.00 sec)
Rows matched: 1 Changed: 1 Warnings: 0

mysql> select * from test.t1;
+-----+
| id |
+-----+
| 591 |
+-----+
1 row in set (0.01 sec)

mysql> set autocommit = ON;
Query OK, 0 rows affected (0.01 sec)

mysql> rollback;

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

consistent with mysql
select * from test.t1;
+-----+
| id |
+-----+
| 590 |
+-----+

3. What did you see instead (Required)

select * from test.t1;
+-----+
| id |
+-----+
| 591 |
+-----+
1 row in set (0.01 sec)

4. What is your TiDB version? (Required)

Release Version: v6.1.0
Edition: Community
Git Commit Hash: 1a89dec
Git Branch: heads/refs/tags/v6.1.0
UTC Build Time: 2022-06-05 05:15:11
GoVersion: go1.18.2
Race Enabled: false
TiKV Min Version: v3.0.0-60965b006877ca7234adaced7890d7b029ed1306
Check Table Before Drop: false |

@keeplearning20221 keeplearning20221 added the type/bug This issue is a bug. label Jul 26, 2022
@ti-chi-bot ti-chi-bot added may-affects-4.0 This bug maybe affects 4.0.x versions. may-affects-5.0 This bug maybe affects 5.0.x versions. may-affects-5.1 This bug maybe affects 5.1.x versions. may-affects-5.2 This bug maybe affects 5.2.x versions. may-affects-5.3 This bug maybe affects 5.3.x versions. may-affects-5.4 This bug maybe affects 5.4.x versions. may-affects-6.0 may-affects-6.1 may-affects-6.2 labels Jul 27, 2022
@seiya-annie
Copy link

tested in mysql 8.0.23:

mysql> create table test(a int);
Query OK, 0 rows affected (0.05 sec)

mysql> insert into test values (590);
Query OK, 1 row affected (0.01 sec)

mysql> set autocommit = ON;
Query OK, 0 rows affected (0.01 sec)

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

mysql> select * from test;
+------+
| a    |
+------+
|  590 |
+------+
1 row in set (0.00 sec)

mysql> update test set a= a+1;
Query OK, 1 row affected (0.00 sec)
Rows matched: 1  Changed: 1  Warnings: 0

mysql> select * from test;
+------+
| a    |
+------+
|  591 |
+------+
1 row in set (0.00 sec)

mysql> set autocommit = ON;
Query OK, 0 rows affected (0.00 sec)

mysql> rollback;
Query OK, 0 rows affected (0.01 sec)

mysql> select * from test;
+------+
| a    |
+------+
|  590 |
+------+
1 row in set (0.00 sec)

mysql> exit

@seiya-annie
Copy link

tried on 4.0.16, exists the same issue

@cfzjywxk
Copy link
Contributor

@seiya-annie
Thanks for the analysis, it's needed to be fixed.

@cfzjywxk cfzjywxk self-assigned this Jul 27, 2022
@jebter jebter added affects-4.0 This bug affects 4.0.x versions. affects-5.0 This bug affects 5.0.x versions. affects-5.1 This bug affects 5.1.x versions. affects-5.2 This bug affects 5.2.x versions. affects-5.3 This bug affects 5.3.x versions. affects-5.4 This bug affects 5.4.x versions. affects-6.0 affects-6.1 affects-6.2 labels Jul 28, 2022
@ti-chi-bot ti-chi-bot removed may-affects-5.0 This bug maybe affects 5.0.x versions. may-affects-5.1 This bug maybe affects 5.1.x versions. may-affects-4.0 This bug maybe affects 4.0.x versions. may-affects-5.4 This bug maybe affects 5.4.x versions. labels Jul 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects-4.0 This bug affects 4.0.x versions. affects-5.0 This bug affects 5.0.x versions. affects-5.1 This bug affects 5.1.x versions. affects-5.2 This bug affects 5.2.x versions. affects-5.3 This bug affects 5.3.x versions. affects-5.4 This bug affects 5.4.x versions. affects-6.0 affects-6.1 affects-6.2 severity/major sig/transaction SIG:Transaction type/bug This issue is a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants