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));
       }

Reply via email to