DonnyZone opened a new pull request #1904: [CALCITE-3893] [CALCITE-3895] SQL with GROUP_ID may generate wrong plan URL: https://github.com/apache/calcite/pull/1904 This PR targets on [CALCITE-3893](https://issues.apache.org/jira/browse/CALCITE-3893) and CALCITE-3895(https://issues.apache.org/jira/browse/CALCITE-3895). - Add constraint check on `Aggregate`'s `groupSet` and `groupSets`. - When rewriting the `group_id()` into `union-all`, we seperate the `groupSets` to many pieces. However, we should not forget to calculate the `groupSet` accordingly. - Recompute `grouping()` and `grouping_id()` function, which takes `groupSet` as input.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
