(loader): spelling checked && fixed complicated logics

Author: sadwitdastreetz

hugegraph-loader/src/main/java/org/apache/hugegraph/loader/HugeGraphLoader.java

@@ -113,6 +113,7 @@ public static void main(String[] args) {
             loader = new HugeGraphLoader(args);
         } catch (Throwable e) {
             Printer.printError("Failed to start loading", e);
+            System.exit(1);
             return;
         }
 
@@ -178,12 +179,14 @@ private void setGraphMode() {
                 () -> this.mapping.structs().stream().filter(struct -> !struct.skip())
                                   .map(InputStruct::input);
 
-        if (inputsSupplier.get().anyMatch(input -> SourceType.GRAPH.equals(input.type()))) {
-            if (!inputsSupplier.get().allMatch(input -> SourceType.GRAPH.equals(input.type()))) {
-                throw new LoadException("All inputs must be of Graph Type");
-            }
-            this.context().setRestoreMode();
-        } else if (this.options.restore) {
+        boolean allMatch = inputsSupplier.get().allMatch(input -> SourceType.GRAPH.equals(input.type()));
+        boolean anyMatch = inputsSupplier.get().anyMatch(input -> SourceType.GRAPH.equals(input.type()));
+
+        if (anyMatch && !allMatch) {
+            throw new LoadException("All inputs must be of Graph Type");
+        }
+
+        if (allMatch || this.options.restore) {
             this.context().setRestoreMode();
         } else {
             this.context().setLoadingMode();
@@ -219,11 +222,7 @@ public boolean load() {
                 LOG.warn(logMessage);
             }
 
-            RuntimeException e = LoadUtil.targetRuntimeException(t);
-            Printer.printError("Failed to load", e);
-            LOG.error("Load failed with exception", e);
-
-            throw e;
+            throw LoadUtil.targetRuntimeException(t);
         }
 
         return true;
@@ -354,7 +353,7 @@ private void createGraphSourceLabels(
             HugeClient sourceClient,
             HugeClient targetClient,
             List<? extends SchemaLabel> labels, // VertexLabel or EdgeLabel
-            Map<String, GraphSource.SeletedLabelDes> selectedMap,
+            Map<String, GraphSource.SelectedLabelDes> selectedMap,
             Map<String, GraphSource.IgnoredLabelDes> ignoredMap,
             boolean isVertex) {
 
@@ -419,10 +418,10 @@ private void createGraphSourceVertexLabel(HugeClient sourceClient,
             vertexLabels = sourceClient.schema().getVertexLabels();
         }
 
-        Map<String, GraphSource.SeletedLabelDes> mapSelectedVertices
+        Map<String, GraphSource.SelectedLabelDes> mapSelectedVertices
                 = new HashMap<>();
         if (graphSource.getSelectedVertices() != null) {
-            for (GraphSource.SeletedLabelDes des :
+            for (GraphSource.SelectedLabelDes des :
                     graphSource.getSelectedVertices()) {
                 mapSelectedVertices.put(des.getLabel(), des);
             }
@@ -474,10 +473,10 @@ private void createGraphSourceEdgeLabel(HugeClient sourceClient,
             edgeLabels = sourceClient.schema().getEdgeLabels();
         }
 
-        Map<String, GraphSource.SeletedLabelDes> mapSelectedEdges
+        Map<String, GraphSource.SelectedLabelDes> mapSelectedEdges
                 = new HashMap<>();
         if (graphSource.getSelectedEdges() != null) {
-            for (GraphSource.SeletedLabelDes des :
+            for (GraphSource.SelectedLabelDes des :
                     graphSource.getSelectedEdges()) {
                 mapSelectedEdges.put(des.getLabel(), des);
             }

hugegraph-loader/src/main/java/org/apache/hugegraph/loader/builder/EdgeBuilder.java

@@ -46,7 +46,7 @@ public class EdgeBuilder extends ElementBuilder<Edge> {
     private final VertexLabel sourceLabel;
     private final VertexLabel targetLabel;
     private final Collection<String> nonNullKeys;
-    // Used to optimize access performace
+    // Used to optimize access performance
     private VertexIdsIndex vertexIdsIndex;
     private String[] lastNames;
 

hugegraph-loader/src/main/java/org/apache/hugegraph/loader/reader/file/FileLineFetcher.java

@@ -84,7 +84,7 @@ public boolean ready() {
 
     @Override
     public void resetReader() {
-        LOG.error("resetReader called, reader reset to null, offset={}", this.offset());
+        LOG.debug("resetReader called, reader reset to null, offset={}", this.offset());
         this.reader = null;
     }
 

hugegraph-loader/src/main/java/org/apache/hugegraph/loader/reader/graph/GraphReader.java

@@ -99,7 +99,7 @@ public void init(LoadContext context,
         // Do with Vertex
         // 1. Get All Selected Vertex
         if (this.source.getSelectedVertices() != null) {
-            for (GraphSource.SeletedLabelDes selected :
+            for (GraphSource.SelectedLabelDes selected :
                     this.source.getSelectedVertices()) {
 
                 selectedVertices.put(selected.getLabel(), null);
@@ -141,7 +141,7 @@ public void init(LoadContext context,
         // Do with edges
         // 1. Get All Selected Edges
         if (this.source.getSelectedEdges() != null) {
-            for (GraphSource.SeletedLabelDes selected :
+            for (GraphSource.SelectedLabelDes selected :
                     this.source.getSelectedEdges()) {
                 selectedEdges.put(selected.getLabel(), null);
                 if (selected.getQuery() != null && selected.getQuery().size() > 0) {

hugegraph-loader/src/main/java/org/apache/hugegraph/loader/source/graph/GraphSource.java

@@ -56,13 +56,13 @@ public class GraphSource extends AbstractSource {
     private String password;
 
     @JsonProperty("selected_vertices")
-    private List<SeletedLabelDes> selectedVertices;
+    private List<SelectedLabelDes> selectedVertices;
 
     @JsonProperty("ignored_vertices")
     private List<IgnoredLabelDes> ignoredVertices;
 
     @JsonProperty("selected_edges")
-    private List<SeletedLabelDes> selectedEdges;
+    private List<SelectedLabelDes> selectedEdges;
 
     @JsonProperty("ignored_edges")
     private List<IgnoredLabelDes> ignoredEdges;
@@ -97,7 +97,7 @@ public FileSource asFileSource() {
     }
 
     @Data
-    public static class SeletedLabelDes {
+    public static class SelectedLabelDes {
         @JsonProperty("query")
         private Map<String, Object> query;
 

hugegraph-loader/src/main/java/org/apache/hugegraph/loader/util/DataTypeUtil.java

@@ -63,7 +63,7 @@ public static boolean isSimpleValue(Object value) {
     public static Object convert(Object value, PropertyKey propertyKey,
                                  InputSource source) {
         E.checkArgumentNotNull(value, "The value of Property(%s) to be " +
-                                      "converted. can't be null", propertyKey.name());
+                                      "converted can't be null", propertyKey.name());
 
         String key = propertyKey.name();
         DataType dataType = propertyKey.dataType();

hugegraph-loader/src/main/java/org/apache/hugegraph/loader/util/HugeClientHolder.java

@@ -152,7 +152,7 @@ protected static void pickHostFromMeta(LoadOptions options) {
         List<String> urls = clientFactory.getAutoURLs(options.cluster,
                                                       options.graphSpace, null);
 
-        E.checkState(CollectionUtils.isNotEmpty(urls), "No avaliable service!");
+        E.checkState(CollectionUtils.isNotEmpty(urls), "No available service!");
 
         int r = (int) Math.floor(Math.random() * urls.size());
         String url = urls.get(r);