[
https://issues.apache.org/jira/browse/BEAM-4702?focusedWorklogId=118008&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-118008
]
ASF GitHub Bot logged work on BEAM-4702:
----------------------------------------
Author: ASF GitHub Bot
Created on: 01/Jul/18 22:07
Start Date: 01/Jul/18 22:07
Worklog Time Spent: 10m
Work Description: kennknowles commented on a change in pull request
#5850: [BEAM-4702] Re-window into global window after GROUP BY <window> plus a
couple other little fixes
URL: https://github.com/apache/beam/pull/5850#discussion_r199363080
##########
File path:
sdks/java/extensions/sql/src/main/java/org/apache/beam/sdk/extensions/sql/impl/transform/BeamAggregationTransforms.java
##########
@@ -185,6 +185,7 @@ public AggregationAdaptor(
aggregators.add(BeamBuiltinAggregations.createMin(call.type.getSqlTypeName()));
break;
case "SUM":
+ case "$SUM0":
Review comment:
This one raises the issue that we maybe should start our test coverage of
operators with a bunch of these internal variants that Beam SQL may already
support, at least partly.
(this one is probably a hack)
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 118008)
Time Spent: 0.5h (was: 20m)
> After SQL GROUP BY <windowing> the result should be globally windowed
> ---------------------------------------------------------------------
>
> Key: BEAM-4702
> URL: https://issues.apache.org/jira/browse/BEAM-4702
> Project: Beam
> Issue Type: New Feature
> Components: dsl-sql
> Reporter: Kenneth Knowles
> Assignee: Kenneth Knowles
> Priority: Major
> Time Spent: 0.5h
> Remaining Estimate: 0h
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
