[MCOL-3307] Non-Columnstore Window function causes debug assert Created: 2019-05-13 Updated: 2020-04-02 Resolved: 2019-05-20 |
|
| Status: | Closed |
| Project: | MariaDB ColumnStore |
| Component/s: | MariaDB Server |
| Affects Version/s: | 1.1.7 |
| Fix Version/s: | 1.1.0, 1.2.4 |
| Type: | Bug | Priority: | Major |
| Reporter: | David Hall (Inactive) | Assignee: | Daniel Lee (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||
| Sprint: | 2019-05 | ||||||||
| Description |
|
CREATE TABLE s3 ( SELECT k1, c1, avg(c1) OVER (ORDER BY c1) FROM s3 WHERE k1 < 100 ORDER BY k1; Cause assert see |
| Comments |
| Comment by David Hall (Inactive) [ 2019-05-13 ] |
|
The problem is that we try to run the query as Columnstore, even though there's no columnstore table involved. During this process, we execute: // @InfiniDB We don't need tmp table for vtable create phase. Plus in the optimizer. This has the effect of setting things up such that the assert happens during this Columnstore attempt. Not sure if the best thing to do is remove the assert (things appear to work without it) or something else. |
| Comment by David Hall (Inactive) [ 2019-05-13 ] |
|
After some gdb poking, the best thing to do is not do the assert during the initial Columnstore attempt. |
| Comment by Daniel Lee (Inactive) [ 2019-05-17 ] |
|
Build tested: GitHub source, tag=columnstore-1.1.7 Created a build with debug flag and reproduced the issue [root@localhost ~]# mcsmysql mytest Copyright (c) 2000, 2018, Oracle, MariaDB Corporation Ab and others. Type 'help;' or '\h' for help. Type '\c' to clear the current input statement. MariaDB [mytest]> CREATE TABLE s3 ( MariaDB [mytest]> SELECT k1, c1, avg(c1) OVER (ORDER BY c1) FROM s3 WHERE k1 < 100 ORDER BY k1; |
| Comment by Daniel Lee (Inactive) [ 2019-05-17 ] |
|
Build verified: Github 1.1.8-1 source /root/columnstore/mariadb-columnstore-server Merge pull request #159 from mariadb-corporation/ /root/columnstore/mariadb-columnstore-server/mariadb-columnstore-engine Merge pull request #766 from mariadb-corporation/ Made build from the latest develop-1.1 source with debug flag and verified that assertion no longer occurs. [root@localhost ~]# mcsmysql mytest Copyright (c) 2000, 2018, Oracle, MariaDB Corporation Ab and others. Type 'help;' or '\h' for help. Type '\c' to clear the current input statement. MariaDB [mytest]> CREATE TABLE s3 ( MariaDB [mytest]> SELECT k1, c1, avg(c1) OVER (ORDER BY c1) FROM s3 WHERE k1 < 100 ORDER BY k1; MariaDB [mytest]> quit Still waiting for 1.2.4-1 source to merge up. |
| Comment by Daniel Lee (Inactive) [ 2019-05-20 ] |
|
Build verified: 1.2.4-1 GitHub source /root/columnstore/mariadb-columnstore-server Merge pull request #161 from mariadb-corporation/ Fix warning during RPM install /root/columnstore/mariadb-columnstore-server/mariadb-columnstore-engine Merge pull request #767 from mariadb-corporation/develop-1.2-merge-up-20190517 Merge develop-1.1 into develop-1.2 |