[GitHub] ivankelly commented on issue #1355: Remove as many Thread.sleep calls from REST endpoint

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

ivankelly commented on issue #1355: Remove as many Thread.sleep calls from REST endpoint
URL: https://github.com/apache/incubator-pulsar/pull/1355#issuecomment-371434684
 
 
   @merlimat @mgodave 
   > The real problem is that in ZK there is no read-your-write consistency by default. 
   
   There is read-your-writes within a session. However, which zookeeper client cache thing seems to be able to handle some errors by creating a new client.  And it's caching data, only updating with watchers. There's no guarantee that when your create succeeds the watchers will have run. Quite the opposite in fact, as the server will respond with success for the write first and then trigger the watchers.
   
   I think the sleep is trying to solve a different problem though. It's not to ensure that the local client has seen update, but to try to ensure that all brokers have seen it, which is perfectly natural in distsys.

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