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

feat: transactions #258

Closed
wants to merge 3 commits into from
Closed

Conversation

morigs
Copy link

@morigs morigs commented Feb 12, 2023

Resolves #253

@zcourts
Copy link

zcourts commented May 11, 2023

Hi @morigs
Is there a blocker preventing this moving forward?

We're forced to use the Java client at the moment because tx support isn't widely available in Rust or C++ clients yet but it'll make things a lot better for us to merge that into our rust impl and drop the Java service which only exist because of this missing feature.

@morigs
Copy link
Author

morigs commented May 11, 2023

@zcourts
No blocker. Adding transactions support is in fact a big epic and requires a lot of effort.
I'm not that familiar with pulsar, so it's better for some of the contributors to decompose the task. In that case me and other contributors will be able to do separate isolated issues.
Unfortunately, I don't have enough time and expertise to do all this myself :(

@morigs morigs closed this Jun 9, 2023
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.

Transaction API support
2 participants