Add more defaults to ONDEMAND

Author: DomGarguilo

core/src/main/java/org/apache/accumulo/core/clientImpl/ImportConfigurationImpl.java

@@ -48,8 +48,8 @@ public Builder setKeepMappings(boolean keepMappings) {
 
   @Override
   public Builder setInitialHostingGoal(TabletHostingGoal initialTabletHostingGoal) {
-    Preconditions.checkState(initialTabletHostingGoal != null, "initialTabletHostingGoal is null");
     Preconditions.checkState(!built, BUILT_ERROR_MSG);
+    Preconditions.checkState(initialTabletHostingGoal != null, "initialTabletHostingGoal is null");
     this.initialHostingGoal = initialTabletHostingGoal;
     return this;
   }

server/manager/src/main/java/org/apache/accumulo/manager/tableOps/tableImport/ImportTable.java

@@ -50,6 +50,8 @@
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
+import com.google.common.base.Preconditions;
+
 import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 
 /**
@@ -71,6 +73,7 @@ public ImportTable(String user, String tableName, Set<String> exportDirs, Namesp
     tableInfo.directories = parseExportDir(exportDirs);
     tableInfo.keepMappings = keepMappings;
     tableInfo.keepOffline = keepOffline;
+    Preconditions.checkState(initialHostingGoal != null, "initialHostingGoal is null");
     tableInfo.initialHostingGoal = initialHostingGoal;
   }
 

server/manager/src/main/java/org/apache/accumulo/manager/tableOps/tableImport/ImportedTableInfo.java

@@ -37,7 +37,7 @@ class ImportedTableInfo implements Serializable {
   public String exportFile;
   public boolean keepMappings;
   public boolean keepOffline;
-  public TabletHostingGoal initialHostingGoal;
+  public TabletHostingGoal initialHostingGoal = TabletHostingGoal.ONDEMAND;
 
   static class DirectoryMapping implements Serializable {
     private static final long serialVersionUID = 1L;