Description
AsyncCuratorFrameworkDsl AsyncCuratorFramework.with(UnhandledErrorListener listener) can be used to construct a new framework level object with a global-like error listener. I may be ok for a operation level error listener(e.g. ErrorListenerPathable) to swallow exception from BackgroundCallback but it is too crude to swallow exceptions in global level.
The call chain is similar to async.with(listener).create().forPath("/foo"). From this, I don't think we are binding an error listener for create() only.
Besides this, I think it is a good to complete CompletableFuture in all cases.
Attachments
Issue Links
- relates to
-
CURATOR-673 Complete BackgroundCallback if curator got closed or exceptions from no-zookeeper world
- Resolved
- links to