[MCOL-4766] UPDATE/INSERT in a transaction does not revert back extent ranges on a rollback Created: 2021-06-18 Updated: 2021-07-09 Resolved: 2021-07-09 |
|
| Status: | Closed |
| Project: | MariaDB ColumnStore |
| Component/s: | None |
| Affects Version/s: | 6.1.1 |
| Fix Version/s: | 6.1.1 |
| Type: | Bug | Priority: | Blocker |
| Reporter: | Gagan Goel (Inactive) | Assignee: | Daniel Lee (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | regression | ||
| Issue Links: |
|
||||||||||||
| Sprint: | 2021-9 | ||||||||||||
| Description |
|
If an UPDATE statement is performed in a transaction (autocommit=off), the extent ranges are updated to reflect the results of the UPDATE operation. However, if the transaction is rolled-back, the extent ranges are not-reverted back, causing simple WHERE predicates to fail. Here are steps to reproduce:
The below incorrectly returns an empty set. The extent ranges are also out-of-sync after the rollback, which is the cause of the WHERE predicate failing.
Same is true for an INSERT in a transation; the extent ranges are out-of-sync with the actual data in the table on a rollback:
As can be seen above, the min/max values of the extent are incorrectly set to -1 and 100. Correct values should be 1 and 2 respectively. |
| Comments |
| Comment by Sergey Zefirov [ 2021-06-23 ] | ||||||||||||||||||||||||
|
https://github.com/mariadb-SergeyZefirov/mariadb-columnstore-engine/tree/MCOL-4766-UPDATE-INSERT-in-a-transaction-does-not-revert-back-extent-ranges-on-a-rollback - relevant branch. Right now it appears I fixed that incorrect behavior. I think I need to review and test what I did some more. | ||||||||||||||||||||||||
| Comment by Sergey Zefirov [ 2021-06-23 ] | ||||||||||||||||||||||||
|
My test sequence:
I think I need to create an MTR test for it and think it through. | ||||||||||||||||||||||||
| Comment by Sergey Zefirov [ 2021-07-07 ] | ||||||||||||||||||||||||
|
It appears that I have rectification for that defect. Right now I run MTR tests with it to see what I had broken there. | ||||||||||||||||||||||||
| Comment by Daniel Lee (Inactive) [ 2021-07-09 ] | ||||||||||||||||||||||||
|
Build verified: 611 (#2754) Also reproduced the issue in an earlier build. |