Skip to content

Support removal of watchers from Zookeeper client. #91

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tejas-sangol
Copy link

Enables the application layer to stop watching over the channel provided by the Watch APIs and use these APIs to clean up the "connection.watchers" map. Channels that are no longer watched upon will also be garbage collected with this change.

The expectation is that the application can use context with cancellation along with the channel returned by the Watch APIs and call these APIs when the context is cancelled. The application can choose to call them independently as well.

These functions can also be used to support the removeWatches API that were added to Zookeeper 3.5.0 and above by actually making a call to Zookeeper whenever needed.

Enables the application layer to stop watching over the channel
provided by the Watch APIs and use these APIs to clean up the
"connection.watchers" map. The channels that are no longer watched
upon will also be garbage collected with this change.

The expectation is that the application can use context with cancellation
along with the channel returned by the Watch APIs and call these
when the context is cancelled. Or the application can choose to call them
independently as well.

These functions can also be used to support the removeWatches API
that were added to Zookeeper 3.5.0 and above by actually making a
call to Zookeeper whenever needed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant