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.

Sign up for GitHub

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

Jump to bottom

Implement building blocks of multi-key transactions #45

Open
bonifaido opened this issue Jan 27, 2018 · 1 comment
Open

Implement building blocks of multi-key transactions #45

bonifaido opened this issue Jan 27, 2018 · 1 comment

Comments

@bonifaido
Copy link
Owner

rust-zookeeper would needs transaction support first though (#4)? This correctly we'd need something similar to the Java client's multi method, and possibly something like the Transaction builder - which seems like a wrapper on top of multi. That would also involve creating something similar to Op, and finally, support for serializing and deserializing the multi requests and responses - similar to MultiTransactionRecord and MultiResponse.
@bonifaido bonifaido mentioned this issue Jan 27, 2018
Merged
tgockel added a commit to tgockel/zookeeper-rs that referenced this issue Jan 29, 2018
@tgockel
Issue bonifaido#45: Creates multi-key transactions.
bb1c748 
It always returns Ok because I can't get the serialization of responses working
correctly. I'm looking at the C code for doing the same thing, but it isn't
matching up to my expectations.
@tgockel
Copy link
Contributor

tgockel commented Jan 29, 2018

I have a branch for this here: https://github.com/tgockel/rust-zookeeper/tree/issue/45/transactions . The code doesn't work because I can't get the deserialization of responses working correctly and I have no idea what I'm doing wrong :-(

tgockel added a commit to tgockel/zookeeper-rs that referenced this issue May 29, 2020
@tgockel
Issue bonifaido#45: Creates multi-key transactions.
c63be70 
It always returns Ok because I can't get the serialization of responses working
correctly. I'm looking at the C code for doing the same thing, but it isn't
matching up to my expectations.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bonifaido @tgockel