adamkennedy commented on code in PR #3757:
URL: https://github.com/apache/calcite/pull/3757#discussion_r1561821674
##########
core/src/main/java/org/apache/calcite/tools/RelBuilder.java:
##########
@@ -2503,7 +2503,7 @@ private RelBuilder
pruneAggregateInputFieldsAndDeduplicateAggCalls(
builder.add(project.getRowType().getFieldList().get(i));
}
r =
- project.copy(cluster.traitSet(), project.getInput(), newProjects,
+ project.copy(project.getTraitSet(), project.getInput(),
newProjects,
Review Comment:
Can we always propagate all traits? Since this happens when the fields of an
existing project are pruned down, couldn't a Collation or Distribution trait
end up being set incorrectly?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
