ivankelly commented on a change in pull request #521: Issue 520: Add more http 
endpoint
URL: https://github.com/apache/bookkeeper/pull/521#discussion_r141303859
 
 

 ##########
 File path: 
bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieShell.java
 ##########
 @@ -234,7 +234,7 @@ public BookieFormatCmd() {
             opts.addOption("f", "force", false,
                     "If [nonInteractive] is specified, then whether"
                             + " to force delete the old data without 
prompt..?");
-            opts.addOption("d", "deleteCookie", false, "Delete its cookie on 
zookeeper");
+            opts.addOption("d", "delete_cookie", false, "Delete its cookie on 
zookeeper");
 
 Review comment:
   BC break here. Can you add a hidden option for deleteCookie, which behaves 
the same as delete_cookie but doesn't show up in the help?
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to