Skip to content
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

Transactions support #320

Open
alexbidiuk opened this issue Sep 13, 2022 · 3 comments
Open

Transactions support #320

alexbidiuk opened this issue Sep 13, 2022 · 3 comments

Comments

@alexbidiuk
Copy link

There is no support of Transaction class

Do you have plans of implementing it?

@DavidVujic
Copy link
Collaborator

Hi @alexbidiuk, this looks interesting. What is it used for?

@alexbidiuk
Copy link
Author

@DavidVujic , it's just a wrapper to invoke multiple commands in atomic way.

Usage depends on your business logic needs. For example I used it to remove some subtree, starting from children, because other way race conditions were happening

@DavidVujic
Copy link
Collaborator

Thank you for explaining @alexbidiuk!

Usually, this library is aiming to keep up with the features in the official ZooKeeper C Client. But I agree this kind of feature would be useful, and if it is possible to develop it using Node.js (wrapping the basic C Client functionality) I think it would be a great addition.

I'll add the feature request label to this issue. It might also be related to the feature request described in #229.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants