[MCOL-662] Unexpected results in cross engine join Created: 2017-04-12 Updated: 2017-10-25 Resolved: 2017-10-25 |
|
| Status: | Closed |
| Project: | MariaDB ColumnStore |
| Component/s: | ExeMgr |
| Affects Version/s: | 1.0.8 |
| Fix Version/s: | 1.0.12, 1.1.1 |
| Type: | Bug | Priority: | Minor |
| Reporter: | Geoff Cleaves | Assignee: | Daniel Lee (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Environment: |
CentOS 7 |
||
| Attachments: |
|
| Sprint: | 2017-9, 2017-20, 2017-21 |
| Description |
|
The following query produces unexpected results. The predicates are acting as an OR instead of AND.
Results:
Expected results:
The query is produced by Tableau and cannot be tweaked. mysqldump file attached in order to create example tables. |
| Comments |
| Comment by Andrew Hutchings (Inactive) [ 2017-04-12 ] |
|
The problem appears to be that the second function in the where condition doesn't appear to be executed at all. I shall look into this further as soon as possible. |
| Comment by David Thompson (Inactive) [ 2017-04-13 ] |
|
Also to confirm this is isolated to the cross engine capability so a possible workaround is to make dim_test a columnstore table. Thanks for filing! |
| Comment by Andrew Hutchings (Inactive) [ 2017-10-23 ] |
|
Filters were overwriting each other in CrossEngine step. Fix was to store the filters in a vector. For QA: description has a pretty good test case (attachment has the tables and data) |
| Comment by Daniel Lee (Inactive) [ 2017-10-24 ] |
|
Build verified: Github source for 1.1.1-1 /root/columnstore/mariadb-columnstore-server Merge pull request #72 from mariadb-corporation/ /root/columnstore/mariadb-columnstore-server/mariadb-columnstore-engine removed duplicaue entries |
| Comment by Daniel Lee (Inactive) [ 2017-10-25 ] |
|
Build verified: Github source for 1.0.12-1 /root/columnstore/mariadb-columnstore-server Merge pull request #69 from mariadb-corporation/ /root/columnstore/mariadb-columnstore-server/mariadb-columnstore-engine Merge pull request #300 from mariadb-corporation/ Verified with same tests |