You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hugegraph.apache.org by GitBox <gi...@apache.org> on 2022/08/17 08:00:12 UTC

[GitHub] [incubator-hugegraph-toolchain] imbajin commented on a diff in pull request #325: fix: Line check argument show detail info

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


##########
hugegraph-loader/src/main/java/com/baidu/hugegraph/loader/reader/line/Line.java:
##########
@@ -37,7 +38,9 @@ public Line(String rawLine, String[] names, Object[] values) {
         E.checkArgumentNotNull(names, "The names can't be null");
         E.checkArgumentNotNull(values, "The values can't be null");
         E.checkArgument(names.length == values.length,
-                        "The length of names %s should be same as values %s");
+                "The length of names %s should be same as values %s",
+                Arrays.toString(names),
+                Arrays.toString(values));

Review Comment:
   please keep the original align style, thanks
   
   and better to show/paste the difference in comment



-- 
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: commits-unsubscribe@hugegraph.apache.org

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