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