wenbingshen opened a new pull request, #3423:
URL: https://github.com/apache/bookkeeper/pull/3423

   ### Motivation
   
   1.`waitForReadOnlyBookie` adds another `listener` thread to observe the node 
status of bookie, which may be out of sync with the triggering of node changes 
in `EnsemblePlacementPolicy`.This `sequence` leads to flaky test. 
   
   2.Expose bookie nodes in EnsemblePlacementPolicy to client.
   
   3.Just change from watching zk to thread sleep.
   
   We currently don't have any additional requirements for getting bookie nodes 
on the client side, so it's better to just use thread sleep here. 
   
   ### Changes
   Change from `waitForReadOnlyBookie` and `waitForWritableBookie` to thread 
`sleep`.
   3 seconds is enough for most.
   
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to