[MCOL-4652] Support for wide decimals in cpimport communication with DBRM Created: 2021-03-30 Updated: 2021-06-18 Resolved: 2021-06-18 |
|
| Status: | Closed |
| Project: | MariaDB ColumnStore |
| Component/s: | None |
| Affects Version/s: | N/A |
| Fix Version/s: | 6.1.1 |
| Type: | Bug | Priority: | Blocker |
| Reporter: | Sergey Zefirov | Assignee: | Daniel Lee (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||
| Sprint: | 2021-5, 2021-6, 2021-7, 2021-8, 2021-9 | ||||||||
| Description |
|
cpimport, when loading wide decimals creates corrupted data which will cause incorrect extent elimination. The root cause is the failure of cpimport to properly communicate with DBRM. The code change for cpimport is accompanied by test scripts to verify the feature (see Pull Request) |
| Comments |
| Comment by Sergey Zefirov [ 2021-03-31 ] |
|
Relevant branch: https://github.com/mariadb-SergeyZefirov/mariadb-columnstore-engine/tree/MCOL-4652-Support-for-wide-decimals-in-cpimport-communication-with-DBRM Contains test and code changes needed for test to pass. May be useful for performance tests on wide decimal columns joins. |
| Comment by Sergey Zefirov [ 2021-04-02 ] |
|
Relevant PR is this: https://github.com/mariadb-corporation/mariadb-columnstore-engine/pull/1840 |
| Comment by Roman [ 2021-05-05 ] |
|
JFYI tntnatbry I am re-taking the issue for review. |
| Comment by Daniel Lee (Inactive) [ 2021-06-17 ] |
|
The PR was declined. Is this ready for testing? |
| Comment by Roman [ 2021-06-18 ] |
|
dleeyh You might had missed the second PR that was merged. |
| Comment by Daniel Lee (Inactive) [ 2021-06-18 ] |
|
I did not miss the fact that PR #1956 was merged on June 1st. PR #1840 was declined a day later. Confirmed with drrtuythat PR #1840 will be ignored and the ticket is ready for testing. |
| Comment by Daniel Lee (Inactive) [ 2021-06-18 ] |
|
Build verified: 6.1.1 ( Drone #2543 ) |