[ 
https://issues.apache.org/jira/browse/BEAM-8317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16938980#comment-16938980
 ] 

Brian Hulette commented on BEAM-8317:
-------------------------------------

Discussed this with [~apilloud]. He said that probably the appropriate fix is 
to make 
[BeamBasicAggregationRule|https://github.com/apache/beam/blob/master/sdks/java/extensions/sql/src/main/java/org/apache/beam/sdk/extensions/sql/impl/rule/BeamBasicAggregationRule.java]
 accept any thing that's _not_ a projection, rather than the current approach 
that just accepts table scans.

Basically it should complement 
[BeamAggregationRule|https://github.com/apache/beam/blob/master/sdks/java/extensions/sql/src/main/java/org/apache/beam/sdk/extensions/sql/impl/rule/BeamAggregationRule.java]
 which accepts _only_ projections.

> SqlTransform doesn't support aggregation over a filter node
> -----------------------------------------------------------
>
>                 Key: BEAM-8317
>                 URL: https://issues.apache.org/jira/browse/BEAM-8317
>             Project: Beam
>          Issue Type: Bug
>          Components: dsl-sql
>    Affects Versions: 2.15.0
>            Reporter: Brian Hulette
>            Priority: Major
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> For example, the following query fails to translate to a physical plan:
> SELECT SUM(f_intValue) FROM PCOLLECTION WHERE f_intValue < 5 GROUP BY 
> f_intGroupingKey



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to