[MCOL-3716] Wrong min/max after cpimport which causes wrong select results Created: 2020-01-09 Updated: 2020-11-12 Resolved: 2020-02-10 |
|
| Status: | Closed |
| Project: | MariaDB ColumnStore |
| Component/s: | cpimport |
| Affects Version/s: | 1.2.5 |
| Fix Version/s: | 1.2.6, 1.4.3, 1.5.1 |
| Type: | Bug | Priority: | Major |
| Reporter: | Richard Stracke | Assignee: | Daniel Lee (Inactive) |
| Resolution: | Fixed | Votes: | 2 |
| Labels: | bulk | ||
| Sprint: | 2020-1, 2020-2 |
| Description |
|
If min/max is set cpimport not delete min/max like another write statement and This cause wrong select results , because extent elimination wrongly exclude extents. Create 2 Files 1 File c1.txt with content "2019-12" 1 File c2.txt with content "2020-01" Execute:
Now load file c1.
Delete and reimport via cpimport will fix it.
If you do the same with a select after the delete, the reimport will fail.
reimport with cpimport file c2.txt as above and:
|
| Comments |
| Comment by David Hall (Inactive) [ 2020-01-10 ] |
|
Appears to break for char columns. Int Columns behave correctly. |
| Comment by David Hall (Inactive) [ 2020-01-20 ] |
|
Tested in: |
| Comment by Patrick LeBlanc (Inactive) [ 2020-01-24 ] |
|
Created the PR against develop. Will down-merge into dev-1.2 and 1.4 before the next release. |
| Comment by Patrick LeBlanc (Inactive) [ 2020-01-24 ] |
|
Oops. Moved the ticket to testing by mistake. |
| Comment by Patrick LeBlanc (Inactive) [ 2020-01-24 ] |
|
Created a PR for the regression test. Used the sequence I specified above. Verified the test catches the problem. |
| Comment by Andrew Hutchings (Inactive) [ 2020-01-27 ] |
|
pleblanc should this be marked as "In Review"? |
| Comment by Daniel Lee (Inactive) [ 2020-02-10 ] |
|
Build verified: 1.4.3-1 source server engine Reproduced the issue in 1.2.5-1 and verified fix in 1.4.3-1 |
| Comment by Daniel Lee (Inactive) [ 2020-02-10 ] |
|
Build verified: 1.2.6-1 source server engine |
| Comment by Daniel Lee (Inactive) [ 2020-02-10 ] |
|
Build verified: 1.5.0-1 source server engine |