SQOOP-3368: Add fail-fast scenarios to S3 incremental import use cases without -...
authorBoglarka Egyed <bogi@apache.org>
Fri, 31 Aug 2018 07:33:46 +0000 (09:33 +0200)
committerBoglarka Egyed <bogi@apache.org>
Fri, 31 Aug 2018 07:33:46 +0000 (09:33 +0200)
(Boglarka Egyed)

src/java/org/apache/sqoop/tool/ImportTool.java
src/test/org/apache/sqoop/tool/TestS3IncrementalImportOptionValidations.java [new file with mode: 0644]

index 1397337..4d5d0d3 100644 (file)
@@ -59,6 +59,7 @@ import org.apache.sqoop.util.ClassLoaderStack;
 import org.apache.sqoop.util.ImportException;
 
 import static org.apache.sqoop.manager.SupportedManagers.MYSQL;
+import static org.apache.commons.lang3.StringUtils.startsWith;
 
 /**
  * Tool that performs database imports to HDFS.
@@ -84,6 +85,8 @@ public class ImportTool extends BaseSqoopTool {
 
   private final HiveClientFactory hiveClientFactory;
 
+  private final String S3_URI_SCHEME = "s3a://";
+
   public ImportTool() {
     this("import", false);
   }
@@ -1166,6 +1169,18 @@ public class ImportTool extends BaseSqoopTool {
           + INCREMENT_TYPE_ARG + " lastmodified cannot be used in conjunction with --"
           + FMT_AVRODATAFILE_ARG + "." + HELP_STR);
     }
+
+    if (isIncrementalModeAppendOrLastmodified(options)
+            && startsWith(options.getTargetDir(), S3_URI_SCHEME)
+            && !startsWith(options.getTempRootDir(), S3_URI_SCHEME)) {
+      throw new InvalidOptionsException("For an " + INCREMENT_TYPE_ARG + " import into an S3 bucket --"
+              + TEMP_ROOTDIR_ARG  + " option must be always set to a location in S3.");
+    }
+  }
+
+  private boolean isIncrementalModeAppendOrLastmodified(SqoopOptions options) {
+    return options.getIncrementalMode() == SqoopOptions.IncrementalMode.AppendRows
+            || options.getIncrementalMode() == SqoopOptions.IncrementalMode.DateLastModified;
   }
 
   @Override
diff --git a/src/test/org/apache/sqoop/tool/TestS3IncrementalImportOptionValidations.java b/src/test/org/apache/sqoop/tool/TestS3IncrementalImportOptionValidations.java
new file mode 100644 (file)
index 0000000..7745f1b
--- /dev/null
@@ -0,0 +1,108 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.sqoop.tool;
+
+import org.apache.sqoop.SqoopOptions;
+import org.junit.Before;
+import org.junit.Rule;
+import org.junit.Test;
+import org.junit.rules.ExpectedException;
+import org.junit.runner.RunWith;
+import org.junit.runners.Parameterized;
+
+import java.util.Arrays;
+import java.util.Properties;
+
+import static org.apache.sqoop.SqoopOptions.IncrementalMode.AppendRows;
+import static org.apache.sqoop.SqoopOptions.IncrementalMode.DateLastModified;
+import static org.apache.sqoop.tool.BaseSqoopTool.INCREMENT_TYPE_ARG;
+import static org.apache.sqoop.tool.BaseSqoopTool.TEMP_ROOTDIR_ARG;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.when;
+
+@RunWith(Parameterized.class)
+public class TestS3IncrementalImportOptionValidations {
+
+    @Parameterized.Parameters(name = "incrementalMode = {0}")
+    public static Iterable<? extends Object> parameters() {
+        return Arrays.asList(AppendRows, DateLastModified);
+    }
+
+    private static final String TEST_TABLE = "testtable";
+    private static final String TEST_CONNECTION_STRING = "testconnectstring";
+    private static final String TEST_TARGET_DIR = "s3a://test-bucket";
+    private static final String TEST_NOT_S3_TEMPORARY_ROOTDIR = "file:///test_temporary_rootdir";
+    private static final String TEST_S3_TEMPORARY_ROOTDIR = "s3a://test_temporary_rootdir";
+    private static final String TEST_CHECK_COLUMN = "testcheckcolumn";
+
+    @Rule
+    public ExpectedException expectedException = ExpectedException.none();
+
+    private final SqoopOptions.IncrementalMode incrementalMode;
+
+    private SqoopOptions sqoopOptions;
+
+    private ImportTool importTool;
+
+    public TestS3IncrementalImportOptionValidations(SqoopOptions.IncrementalMode incrementalMode) {this.incrementalMode = incrementalMode;}
+
+    @Before
+    public void before() {
+        sqoopOptions = mock(SqoopOptions.class);
+        when(sqoopOptions.getTableName()).thenReturn(TEST_TABLE);
+        when(sqoopOptions.getConnectString()).thenReturn(TEST_CONNECTION_STRING);
+        when(sqoopOptions.getTargetDir()).thenReturn(TEST_TARGET_DIR);
+        when(sqoopOptions.getIncrementalTestColumn()).thenReturn(TEST_CHECK_COLUMN);
+        when(sqoopOptions.getMapColumnHive()).thenReturn(new Properties());
+
+        importTool = new ImportTool();
+        importTool.extraArguments = new String[0];
+    }
+
+    @Test
+    public void testValidateOptionsThrowsWhenS3IncrementalImportIsPerformedWithoutTemporaryRootdir() throws Exception {
+        expectedException.expect(SqoopOptions.InvalidOptionsException.class);
+        expectedException.expectMessage("For an " + INCREMENT_TYPE_ARG + " import into an S3 bucket --"
+                + TEMP_ROOTDIR_ARG + " option must be always set to a location in S3.");
+
+        when(sqoopOptions.getIncrementalMode()).thenReturn(incrementalMode);
+
+        importTool.validateOptions(sqoopOptions);
+    }
+
+    @Test
+    public void testValidateOptionsThrowsWhenS3IncrementalImportIsPerformedWithNotS3TemporaryRootdir() throws Exception {
+        expectedException.expect(SqoopOptions.InvalidOptionsException.class);
+        expectedException.expectMessage("For an " + INCREMENT_TYPE_ARG + " import into an S3 bucket --"
+                + TEMP_ROOTDIR_ARG + " option must be always set to a location in S3.");
+
+        when(sqoopOptions.getIncrementalMode()).thenReturn(incrementalMode);
+        when(sqoopOptions.getTempRootDir()).thenReturn(TEST_NOT_S3_TEMPORARY_ROOTDIR);
+
+        importTool.validateOptions(sqoopOptions);
+    }
+
+    @Test
+    public void testValidateOptionsSucceedsWhenS3IncrementalImportIsPerformedWithS3TemporaryRootdir() throws Exception {
+        when(sqoopOptions.getIncrementalMode()).thenReturn(incrementalMode);
+        when(sqoopOptions.getTempRootDir()).thenReturn(TEST_S3_TEMPORARY_ROOTDIR);
+
+        importTool.validateOptions(sqoopOptions);
+    }
+}