[GitHub] [accumulo] ctubbsii opened a new pull request #2434: Add trivial ZK Watcher to MiniAccumuloClusterImpl

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

ctubbsii opened a new pull request #2434:
URL: https://github.com/apache/accumulo/pull/2434


   Add a trivial watcher to MiniAccumuloClusterImpl.verifyUp to avoid an
   annoying ERROR coming from ZooKeeper that is logged when a ZooKeeper
   client is constructed with a null Watcher.
   
   Also address comments on #2414 to remove the redundant use of
   requireNonNull for calls to Optional.get(). Instead, use a private
   helper method to wait for the Process to be recorded as having a start
   time, up to 10 seconds. So, instead of expecting the start time to be
   non-null, it is now expected to be non-null within 10 seconds, and it
   does so using Optional.isEmpty() instead of requireNonNull.
   
   This fixes #2431


-- 
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 #2434: Add trivial ZK Watcher to MiniAccumuloClusterImpl

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

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


   


-- 
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