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

   ### Motivation
    `testWriteSetWriteableCheck` always failed. After investigation, it is 
found that `isWritable=false` is set when the simulated channel is not 
writable, but the unit test cannot fully perceive the channel establishment, 
because the channel is completely established asynchronously. The isWritable 
state of false is changed to true again.
   
   you can see the following logs:
   
![image](https://user-images.githubusercontent.com/35599757/196625314-0ea5a305-9564-4738-8055-20baa6a5c5a2.png)
   
   The smallest change is that we can ensure that the channel is established 
before changing the channel state, such as executing `bkc.getBookieInfo()`, and 
then `setTargetChannelState`
   


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