athanatos commented on issue #1747: Set ConnectionExpired Listener to MetadataClientDriver in AR URL: https://github.com/apache/bookkeeper/pull/1747#issuecomment-428700988 Generally speaking, the transparent reconnect logic in ZooKeeperClient is dangerous in two ways: 1) If the user isn't aware of what's going on, ephemeral nodes and watches can silently disappear. 2) Writes are generally non-idempotent, so retrying them can result in erroneous error conditions, as with node creation returning an EEXIST when it actually succeeded. It's easy to reproduce errors like 2 by introducing intermittent packet drops while creating ledgers.
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
