Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16642 )
Change subject: [tools] mention whitespace as a separator for variadic args ...................................................................... Patch Set 2: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/16642/2//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/16642/2//COMMIT_MSG@10 PS2, Line 10: As it turned out, the : help/usage message In this particular case, was the user actually checking the help message to understand the issue? Regardless, I suppose whatever error message they saw wasn't clear enough for them to try with whitespaces instead of commas. Maybe we should verify the args, especially if we know they are going to be UUIDs. -- To view, visit http://gerrit.cloudera.org:8080/16642 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I3ef979bd17655b5d46e1adb2a2f22923353ac435 Gerrit-Change-Number: 16642 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Fri, 23 Oct 2020 23:30:25 +0000 Gerrit-HasComments: Yes
