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

Jesus Camacho Rodriguez commented on CALCITE-931:
-------------------------------------------------

[~maryannxue], the change makes sense; I just checked the Sort operator 
constructor and we enforce (via assertion) that the traits contain the given 
collation, so I'm wondering why we didn't catch this? 

> Wrong collation trait in SortJoinTransposeRule for right joins.
> ---------------------------------------------------------------
>
>                 Key: CALCITE-931
>                 URL: https://issues.apache.org/jira/browse/CALCITE-931
>             Project: Calcite
>          Issue Type: Bug
>            Reporter: Maryann Xue
>            Assignee: Maryann Xue
>            Priority: Minor
>         Attachments: CALCITE-931.patch
>
>
> The traitSet of the new Sort for Join's right child should be replaced with 
> rightCollation.
> {code}
>       newRightInput = sort.copy(sort.getTraitSet(), join.getRight(), 
> rightCollation,
>           sort.offset, sort.fetch);
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to