From b07906a2a2fc9c5d2808cb53acf1e855cfef0314 Mon Sep 17 00:00:00 2001
From: Kate Ting <kathleen@apache.org>
Date: Thu, 23 May 2013 18:54:23 -0400
Subject: [PATCH] SQOOP-1052: Hive import is not compatible with Avro and
 SequenceFile but no errors are thrown

(Jarek Jarcec Cecho via Kate Ting)
---
 src/java/org/apache/sqoop/tool/BaseSqoopTool.java | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/src/java/org/apache/sqoop/tool/BaseSqoopTool.java b/src/java/org/apache/sqoop/tool/BaseSqoopTool.java
index 9417d57a..42f521f4 100644
--- a/src/java/org/apache/sqoop/tool/BaseSqoopTool.java
+++ b/src/java/org/apache/sqoop/tool/BaseSqoopTool.java
@@ -1050,6 +1050,18 @@ protected void validateHiveOptions(SqoopOptions options)
               + " 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.