[ https://issues.apache.org/jira/browse/CALCITE-5888?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17816983#comment-17816983 ]
Niels Pardon commented on CALCITE-5888: --------------------------------------- As I mentioned in my duplicate issue CALCITE-6261 I do have a fix and will provide a PR soon. > Assertion error in aggregate > ---------------------------- > > Key: CALCITE-5888 > URL: https://issues.apache.org/jira/browse/CALCITE-5888 > Project: Calcite > Issue Type: Bug > Components: core > Affects Versions: 1.35.0, 1.36.0 > Reporter: Petr Masopust > Priority: Major > > We have {{relBuilder.aggregate(groupKey, aggregateCalls)}} in our code with > values {{[AS($31, 'a_label_d_opp_stage_id_foodmart_bd4c9c91212f9f'), > AS(CAST(FLOOR($34, FLAG(QUARTER))):TIMESTAMP(0), > 'a_label_snapshot_timestamp_quarter_foodmart_89fe710c8628d9')]}} and > {{[COUNT(CASE(AND(COALESCE($37, false), =($36, $13)), $15, null:NULL)), > SUM(CASE(AND(COALESCE($37, false), =($36, $13)), $2, null:NULL)), > MAX(CASE(AND(COALESCE($37, false), =($36, $13)), 0, null:NULL)), > MAX(CASE(AND(COALESCE($37, false), =($36, $13)), 0, null:NULL))].}} > It works perfectly in version 1.34.0 but in 1.35.0 we got this exception: > {{java.lang.AssertionError}} > {{ at org.apache.calcite.rel.core.Aggregate.<init>(Aggregate.java:175)}} > {{ at > org.apache.calcite.rel.logical.LogicalAggregate.<init>(LogicalAggregate.java:72)}} > {{ at > org.apache.calcite.rel.logical.LogicalAggregate.create_(LogicalAggregate.java:144)}} > {{ at > org.apache.calcite.rel.logical.LogicalAggregate.create(LogicalAggregate.java:116)}} > {{ at > org.apache.calcite.rel.core.RelFactories$AggregateFactoryImpl.createAggregate(RelFactories.java:328)}} > {{ at > org.apache.calcite.tools.RelBuilder.aggregate_(RelBuilder.java:2564)}} > {{ at > org.apache.calcite.tools.RelBuilder.aggregate_(RelBuilder.java:2511)}} > {{ at org.apache.calcite.tools.RelBuilder.aggregate(RelBuilder.java:2348)}} > > I think it is either missing {{permute}} or assert should compare > {{cardinality}} instead of {{length.}} Because it compares field index? to > number of fields which looks like nonsense to me. -- This message was sent by Atlassian Jira (v8.20.10#820010)