You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by jc...@apache.org on 2018/01/19 17:37:45 UTC

hive git commit: HIVE-18482: Copy-paste error in the RelOptHiveTable (Jesus Camacho Rodriguez, reviewed by Ashutosh Chauhan)

Repository: hive
Updated Branches:
  refs/heads/master d6c6d96f9 -> 41a5c0659


HIVE-18482: Copy-paste error in the RelOptHiveTable (Jesus Camacho Rodriguez, reviewed by Ashutosh Chauhan)


Project: http://git-wip-us.apache.org/repos/asf/hive/repo
Commit: http://git-wip-us.apache.org/repos/asf/hive/commit/41a5c065
Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/41a5c065
Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/41a5c065

Branch: refs/heads/master
Commit: 41a5c06597cfdc29324d3181d4c5e89a93b0aded
Parents: d6c6d96
Author: Jesus Camacho Rodriguez <jc...@apache.org>
Authored: Fri Jan 19 09:37:39 2018 -0800
Committer: Jesus Camacho Rodriguez <jc...@apache.org>
Committed: Fri Jan 19 09:37:39 2018 -0800

----------------------------------------------------------------------
 .../hadoop/hive/ql/optimizer/calcite/RelOptHiveTable.java      | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hive/blob/41a5c065/ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/RelOptHiveTable.java
----------------------------------------------------------------------
diff --git a/ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/RelOptHiveTable.java b/ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/RelOptHiveTable.java
index 23094d1..d0ef0de 100644
--- a/ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/RelOptHiveTable.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/RelOptHiveTable.java
@@ -191,8 +191,7 @@ public class RelOptHiveTable extends RelOptAbstractTable {
             break;
           }
         }
-        if (pkPos == rowType.getFieldNames().size()
-            || pkPos == rowType.getFieldNames().size()) {
+        if (pkPos == rowType.getFieldNames().size()) {
           LOG.error("Column for primary key definition " + pkColName + " not found");
           return ImmutableList.of();
         }
@@ -218,8 +217,7 @@ public class RelOptHiveTable extends RelOptAbstractTable {
             break;
           }
         }
-        if (ukPos == rowType.getFieldNames().size()
-            || ukPos == rowType.getFieldNames().size()) {
+        if (ukPos == rowType.getFieldNames().size()) {
           LOG.error("Column for unique constraint definition " + ukCol.colName + " not found");
           return ImmutableList.of();
         }