sqoop-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From kathl...@apache.org
Subject git commit: SQOOP-1052: Hive import is not compatible with Avro and SequenceFile but no errors are thrown
Date Thu, 23 May 2013 22:57:37 GMT
Updated Branches:
  refs/heads/trunk b2d3a682a -> b07906a2a


SQOOP-1052: Hive import is not compatible with Avro and SequenceFile but no errors are thrown

(Jarek Jarcec Cecho via Kate Ting)


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

Branch: refs/heads/trunk
Commit: b07906a2a2fc9c5d2808cb53acf1e855cfef0314
Parents: b2d3a68
Author: Kate Ting <kathleen@apache.org>
Authored: Thu May 23 18:54:23 2013 -0400
Committer: Kate Ting <kathleen@apache.org>
Committed: Thu May 23 18:54:23 2013 -0400

----------------------------------------------------------------------
 src/java/org/apache/sqoop/tool/BaseSqoopTool.java |   12 ++++++++++++
 1 files changed, 12 insertions(+), 0 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/sqoop/blob/b07906a2/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 9417d57..42f521f 100644
--- a/src/java/org/apache/sqoop/tool/BaseSqoopTool.java
+++ b/src/java/org/apache/sqoop/tool/BaseSqoopTool.java
@@ -1050,6 +1050,18 @@ public abstract class BaseSqoopTool extends com.cloudera.sqoop.tool.SqoopTool
{
               + " option." + HELP_STR);
     }
 
+    if(options.doHiveImport()
+        && options.getFileLayout() == SqoopOptions.FileLayout.AvroDataFile) {
+      throw new InvalidOptionsException("Hive import is not compatible with "
+        + "importing into AVRO format.");
+    }
+
+    if(options.doHiveImport()
+        && options.getFileLayout() == SqoopOptions.FileLayout.SequenceFile) {
+      throw new InvalidOptionsException("Hive import is not compatible with "
+        + "importing into SequenceFile format.");
+    }
+
     // 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.


Mime
View raw message