[jira] [Created] (CALCITE-5412) ROLLUP/CUBE non-standard syntax support logic is buggy

["Leonid Chistov (Jira)" <ji...@apache.org>]

Leonid Chistov created CALCITE-5412:
---------------------------------------

             Summary: ROLLUP/CUBE non-standard syntax support logic is buggy
                 Key: CALCITE-5412
                 URL: https://issues.apache.org/jira/browse/CALCITE-5412
             Project: Calcite
          Issue Type: Bug
    Affects Versions: 1.32.0
            Reporter: Leonid Chistov


There are currently several dialects supported by Calcite which share a logic of using 'GROUP BY ... WITH ROLLUP' or 'GROUP BY ... WITH CUBE' syntax instead of 'GROUP BY ROLLUP(...)' and 'GROUP BY CUBE(...)' syntax: MySQL5, MsSQL, Hive and Spark.

Note: actually Spark should not be on this list anymore, see https://issues.apache.org/jira/browse/CALCITE-5411

Following issues exist regarding current implementation of this logic:
 * There is a bug in Result RelToSqlConverter::visit(Sort e). Check 'hasTrickyRollup' does not take into account possibility of different order of fields in 'ROLLUP' clause and 'ORDER BY' clause. Because of this, we loose information about order of fields in 'ORDER BY' clause in the conversion under this check.
 * CUBE clause seems to be not supported in MYSQL5.x in any form ([https://dev.mysql.com/doc/refman/5.7/en/group-by-modifiers.html]), but we don’t check it. 
 * We don't have an check for the “GROUP BY GROUPING SETS(...)” syntax support, thus we always generate SQL code for such clause, is it was supported



--
This message was sent by Atlassian Jira
(v8.20.10#820010)