You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@beam.apache.org by "Brian Hulette (Jira)" <ji...@apache.org> on 2019/09/26 21:25:00 UTC

[jira] [Commented] (BEAM-8317) SqlTransform doesn't support aggregation over a filter node

    [ 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)