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 <[email protected]> Authored: Thu May 23 18:54:23 2013 -0400 Committer: Kate Ting <[email protected]> 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.
