You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sqoop.apache.org by ja...@apache.org on 2015/04/03 18:12:04 UTC

sqoop git commit: SQOOP-2282: Add validation check for --hive-import and --append

Repository: sqoop
Updated Branches:
  refs/heads/trunk 1a6a83607 -> 64c5fc375


SQOOP-2282: Add validation check for --hive-import and --append

(Abraham Elmahrek via Jarek Jarcec Cecho)


Project: http://git-wip-us.apache.org/repos/asf/sqoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/sqoop/commit/64c5fc37
Tree: http://git-wip-us.apache.org/repos/asf/sqoop/tree/64c5fc37
Diff: http://git-wip-us.apache.org/repos/asf/sqoop/diff/64c5fc37

Branch: refs/heads/trunk
Commit: 64c5fc3755bb26b48f515e1c2da6c5c5249e2b76
Parents: 1a6a836
Author: Jarek Jarcec Cecho <ja...@apache.org>
Authored: Fri Apr 3 09:11:34 2015 -0700
Committer: Jarek Jarcec Cecho <ja...@apache.org>
Committed: Fri Apr 3 09:11:34 2015 -0700

----------------------------------------------------------------------
 src/java/org/apache/sqoop/tool/BaseSqoopTool.java | 6 ++++++
 1 file changed, 6 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/sqoop/blob/64c5fc37/src/java/org/apache/sqoop/tool/BaseSqoopTool.java
----------------------------------------------------------------------
diff --git a/src/java/org/apache/sqoop/tool/BaseSqoopTool.java b/src/java/org/apache/sqoop/tool/BaseSqoopTool.java
index 058e81d..c97bb58 100644
--- a/src/java/org/apache/sqoop/tool/BaseSqoopTool.java
+++ b/src/java/org/apache/sqoop/tool/BaseSqoopTool.java
@@ -1396,6 +1396,12 @@ public abstract class BaseSqoopTool extends com.cloudera.sqoop.tool.SqoopTool {
         + "importing into SequenceFile format.");
     }
 
+    if (options.doHiveImport()
+        && options.isAppendMode()) {
+      throw new InvalidOptionsException("Append mode for hive imports is not "
+          + " yet supported. Please remove the parameter --append-mode");
+    }
+
     // Many users are reporting issues when they are trying to import data
     // directly into hive warehouse. This should prevent users from doing
     // so in case of a default location.