adamkennedy commented on code in PR #3757:
URL: https://github.com/apache/calcite/pull/3757#discussion_r1562780967
##########
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:
I think there might be a slightly better way using Mapping, the column
pruning is effectively a Mapping anyway, so if we were to make it actually a
Mapping there's a method that will transform the TraitSet according to that
mapping, I believe including Collation and Distribution
--
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]
