imbajin commented on code in PR #666:
URL: 
https://github.com/apache/incubator-hugegraph-toolchain/pull/666#discussion_r2126164551


##########
hugegraph-loader/src/main/java/org/apache/hugegraph/loader/executor/LoadOptions.java:
##########
@@ -53,15 +54,17 @@ public class LoadOptions implements Serializable {
                description = "The schema file path which to create manually")
     public String schema;
 
-    @Parameter(names = {"-g", "--graph"}, required = true, arity = 1,
-               description = "The namespace of the graph to load into")
-    public String graph;
+    @Parameter(names = {"-g", "--graph"}, 
+               arity = 1,
+               description = "The namespace of the graph to load into 
(default: DEFAULT)")
+    public String graph = "DEFAULT";

Review Comment:
   @sadwitdastreetz Hi, we need 2 params for it:
   1. `-gs` or `--graphspace`, use `DEFAULT` as the default value, **not 
required**
   2. `-g` or `--graph`, use `hugegraph` as the default value, **not required**\
   
   Also update the description for them



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@hugegraph.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@hugegraph.apache.org
For additional commands, e-mail: issues-h...@hugegraph.apache.org

Reply via email to