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

 ##########
 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:
   Thanks. I C, Since this change is to make a uniform between here and the 
help info in line 252, so would like to discard this change , and change line 
252 into "deleteCookie". so they unified, and not break the old user behavior.
 
----------------------------------------------------------------
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