Skip to content

Commit

Permalink
[CALCITE-4275] EnumerableMergeJoin#create does not set EnumerableConv…
Browse files Browse the repository at this point in the history
…ention in the trait set
  • Loading branch information
rubenada committed Sep 24, 2020
1 parent a8be094 commit 8ca4d86
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,7 @@ protected EnumerableMergeJoin(
Set<CorrelationId> variablesSet,
JoinRelType joinType) {
super(cluster, traits, ImmutableList.of(), left, right, condition, variablesSet, joinType);
assert getConvention() instanceof EnumerableConvention;
final List<RelCollation> leftCollations =
left.getTraitSet().getTraits(RelCollationTraitDef.INSTANCE);
final List<RelCollation> rightCollations =
Expand Down Expand Up @@ -362,7 +363,7 @@ public static EnumerableMergeJoin create(RelNode left, RelNode right,
RexNode condition, ImmutableIntList leftKeys,
ImmutableIntList rightKeys, JoinRelType joinType) {
final RelOptCluster cluster = right.getCluster();
RelTraitSet traitSet = cluster.traitSet();
RelTraitSet traitSet = cluster.traitSetOf(EnumerableConvention.INSTANCE);
if (traitSet.isEnabled(RelCollationTraitDef.INSTANCE)) {
final RelMetadataQuery mq = cluster.getMetadataQuery();
final List<RelCollation> collations =
Expand Down

0 comments on commit 8ca4d86

Please sign in to comment.