You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2022/01/28 10:47:24 UTC

[GitHub] [accumulo] milleruntime opened a new pull request #2438: Refactor Initialize into multiple classes

milleruntime opened a new pull request #2438:
URL: https://github.com/apache/accumulo/pull/2438


   * Create ZooKeeperInitializer
   * Create FileSystemInitializer
   * Create InitialConfiguration
   * Clean up code in Initialize by dropping redundant try/catch blocks,
   improving error handling, passing around new InitialConfiguration object
   to get config properties and improve readability
   * Create new methods in Initialize: checkSASL(), checkUploadProps(), resetSecurity()


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] milleruntime commented on pull request #2438: Refactor Initialize into multiple classes

Posted by GitBox <gi...@apache.org>.

milleruntime commented on pull request #2438:
URL: https://github.com/apache/accumulo/pull/2438#issuecomment-1025727268


   @ctubbsii thanks for the tweaks


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] ctubbsii merged pull request #2438: Refactor Initialize into multiple classes

Posted by GitBox <gi...@apache.org>.

ctubbsii merged pull request #2438:
URL: https://github.com/apache/accumulo/pull/2438


   


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org