[MCOL-4673] Regression: calShowPartition returns N/A Created: 2021-04-09 Updated: 2021-06-18 Resolved: 2021-05-24 |
|
| Status: | Closed |
| Project: | MariaDB ColumnStore |
| Component/s: | ExeMgr |
| Affects Version/s: | 6.1.1 |
| Fix Version/s: | 6.1.1 |
| Type: | Bug | Priority: | Major |
| Reporter: | Alexander Barkov | Assignee: | Sergey Zefirov |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||||||||||||||
| Description |
|
If I compile this ColumnStore revision:
and run this SQL script:
in prints the following output:
Notice, it prints good Min and Max values. If I compile ColumnStore with the next revision in the history:
and rerun the same script, it does not print Min/Max values anymore:
I don't know if this change was intentional. Probably not. |
| Comments |
| Comment by Sergey Zefirov [ 2021-04-20 ] |
|
The problem here is that column's extent is in state "Updating" after an insert operation. It is not "Valid" and is reported as "N/A" in calShowPartitions(). |
| Comment by Sergey Zefirov [ 2021-04-20 ] |
|
https://github.com/mariadb-SergeyZefirov/mariadb-columnstore-engine/tree/MCOL-4673-Regression-calShowPartition-returns-NA - the code that contains a fix. I am not sure about the fix, though, so I postpone pull request. |
| Comment by Sergey Zefirov [ 2021-04-23 ] |
|
Current solution fails columnstore/base.mcs51_cpimport_select_from test (among others, I guess). |
| Comment by Sergey Zefirov [ 2021-04-23 ] |
|
Right now we have two ways to run MTR: with separate DB server and with, say, system-wide DB server (--extern flag). We run MTR with --extern flag in CI. The command line for MTR run with separate server is this: ./mtr --suite=columnstore/basic --skip-test-list=suite/columnstore/basic/failed.def This run fails a lot of tests but does not skip udf_calshowpartitions. The fix I have in source code right now pass this test. I reported failure of mcs51_cpimport_select_from test above and it is from this run. It is not a problem with my patch, but with the way MTR runs. The command line for MTR to run with system-wide server is this: ./mtr --suite=columnstore/basic --extern socket=/var/lib/mysql/mysql.sock --skip-test-list=suite/columnstore/basic/failed.def This run skips udf_calshowpartitions but pass all other tests. I will make sure that cpimport exits with proper extent range set. After that, I will create a PR. |
| Comment by Sergey Zefirov [ 2021-04-27 ] |
|
https://github.com/mariadb-corporation/mariadb-columnstore-engine/pull/1891 - relevant PR. |