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

Haisheng Yuan edited comment on CALCITE-3889 at 4/1/20, 2:16 PM:
-----------------------------------------------------------------

[~rkondakov] There is a fix for the bug you mentioned: 
https://github.com/apache/calcite/pull/1879/files#diff-ef2c263b544b56d503d5332c1d76ef02L144


was (Author: hyuan):
[~rkondakov] There is a fix for the bug you mentioned: 
https://github.com/apache/calcite/pull/1879/files#diff-ef2c263b544b56d503d5332c1d76ef02R144

> Add apply(Mappings.Mapping) to RelTrait and RelTraitSet
> -------------------------------------------------------
>
>                 Key: CALCITE-3889
>                 URL: https://issues.apache.org/jira/browse/CALCITE-3889
>             Project: Calcite
>          Issue Type: Improvement
>          Components: core
>            Reporter: Haisheng Yuan
>            Priority: Major
>
> RelTrait Collation, Distribution have key indices, when we pass down the 
> traitset to child or propagate to parent operator, we have to remap these 
> keys. It would be nice to have {{apply(Mappings.Mapping)}} on RelTrait and 
> RelTraitSet. RelDistribution already has the method, but we may want it on 
> every RelTrait except Convention. 



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

Reply via email to