Open damccorm opened 2 years ago
Unable to assign user @roger-mike. If able, self-assign, otherwise tag @damccorm so that he can assign you. Because of GitHub's spam prevention system, your activity is required to enable assignment in this repo.
Hi @damccorm, could you assign this to me? Thanks
Sure! With https://github.com/apache/beam/pull/21719 merged you should be able to self assign with the command .take-issue
though, so you should be able to do that going forward! Let me know if you run into any issues with that
The OVER clause isn't supported by our ZetaSQL to Calcite translator. It can be trivially enabled in the parser with the example below, but there is some work required to convert the parsed ZetaSQL proto to Calcite logical operators (mostly in AggregateScanConverter).
This is the "over clause" TODO here: https://github.com/apache/beam/blob/243128a8fc52798e1b58b0cf1a271d95ee7aa241/sdks/java/extensions/sql/zetasql/src/main/java/org/apache/beam/sdk/extensions/sql/zetasql/translation/AggregateScanConverter.java#L147
Current state the test fails:
Imported from Jira BEAM-12097. Original Jira may contain additional context. Reported by: apilloud.