smengcl commented on code in PR #5087:
URL: https://github.com/apache/ozone/pull/5087#discussion_r1267464877
##########
hadoop-ozone/tools/src/main/java/org/apache/hadoop/ozone/debug/CompactionLogDagPrinter.java:
##########
@@ -40,14 +40,14 @@
public class CompactionLogDagPrinter extends Handler
implements SubcommandWithParent {
- @CommandLine.Option(names = {"-f", "--file-name"},
- description = "Name of the image file. (optional)")
- private String fileName;
+ @CommandLine.Option(names = {"-f", "--file-name-prefix"},
+ description = "Prefix to be use in image file name. (optional)")
+ private String fileNamePrefix;
// TODO: Change graphType to enum.
@CommandLine.Option(names = {"-t", "--graph-type"},
- description = "Type of node name to use in the graph image.\n" +
- "Accepted values are: \n" +
+ description = "Type of node name to use in the graph image. " +
+ "(optional)\n Accepted values are: \n" +
" file_name: to use file name as node name in DAG,\n" +
Review Comment:
If the default is already indicated by PicoCLI then ignore this 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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
