This is an automated email from the ASF dual-hosted git repository. hyuan pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git
commit 6ee48c66c8a51bfd9734b20927e4c2e919575bc4 Author: cndaimin <[email protected]> AuthorDate: Fri Nov 1 15:27:11 2019 +0800 [CALCITE-3469] Wrong rel used in SubstitutionVisitor#rowTypesAreEquivalent (Min Dai) Close #1555 --- core/src/main/java/org/apache/calcite/plan/SubstitutionVisitor.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/core/src/main/java/org/apache/calcite/plan/SubstitutionVisitor.java b/core/src/main/java/org/apache/calcite/plan/SubstitutionVisitor.java index 0c4e22f..58f57da 100644 --- a/core/src/main/java/org/apache/calcite/plan/SubstitutionVisitor.java +++ b/core/src/main/java/org/apache/calcite/plan/SubstitutionVisitor.java @@ -612,7 +612,7 @@ public class SubstitutionVisitor { return litmus.fail("Mismatch for column count: [{}]", Pair.of(rel0, rel1)); } for (Pair<RelDataTypeField, RelDataTypeField> pair - : Pair.zip(rel0.rowType.getFieldList(), rel0.rowType.getFieldList())) { + : Pair.zip(rel0.rowType.getFieldList(), rel1.rowType.getFieldList())) { if (!pair.left.getType().equals(pair.right.getType())) { return litmus.fail("Mismatch for column type: [{}]", Pair.of(rel0, rel1)); }
