[GitHub] ivankelly commented on a change in pull request #771: Don't require password to create or open ledgers

[GitBox <gi...@apache.org>]

ivankelly commented on a change in pull request #771: Don't require password to create or open ledgers
URL: https://github.com/apache/bookkeeper/pull/771#discussion_r152872587
 
 

 ##########
 File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerCreateOp.java
 ##########
 @@ -222,7 +222,7 @@ private void createComplete(int rc, LedgerHandle lh) {
         private int builderEnsembleSize = 3;
         private int builderAckQuorumSize = 2;
         private int builderWriteQuorumSize = 2;
-        private byte[] builderPassword;
+        private byte[] builderPassword = new byte[0];
 
 Review comment:
   Sure, will update tomorrow, and check the tests too.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services