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

[feature request] Support Transaction API #932

Closed
liangyepianzhou opened this issue Jan 1, 2023 · 3 comments
Closed

[feature request] Support Transaction API #932

liangyepianzhou opened this issue Jan 1, 2023 · 3 comments
Assignees
@liangyepianzhou

Comments

@liangyepianzhou
Copy link
Contributor

liangyepianzhou commented Jan 1, 2023 •
edited
Loading

Motivation

Now, the pulsar-client-go does not support the go-client, so I want to support transaction API for the go client.

Solution

Add Transaction API for go client.
@liangyepianzhou liangyepianzhou self-assigned this Jan 1, 2023
@liangyepianzhou liangyepianzhou mentioned this issue Jan 29, 2023
Closed
1 task
liangyepianzhou added a commit that referenced this issue Feb 13, 2023
@liangyepianzhou
[feat][txn]Implement transactionCoordinatorClient
91803b0 
Master Issue:#932
### Motivation
Implement transaction coordinator client.
### Modifications
1. Implement transaction coordinator
2. implement transaction API
3. Add metric and test
@liangyepianzhou liangyepianzhou mentioned this issue Feb 13, 2023
Merged
1 task
nodece pushed a commit that referenced this issue Feb 27, 2023
@liangyepianzhou
[feat][txn]Implement transactionCoordinatorClient (#953)
e2ea255 
* [feat][txn]Implement transactionCoordinatorClient
Master Issue:#932
### Motivation
Implement transaction coordinator client.
### Modifications
1. Implement transaction coordinator
2. implement transaction API
3. Add metric and test

* Fix checkstyle and CI

* fix some comments

* fix some comments

* fix some comments

* comment style consistent

* comment style consistent

* fix some comments
@liangyepianzhou liangyepianzhou mentioned this issue Mar 7, 2023
Merged
1 task
liangyepianzhou added a commit that referenced this issue Mar 24, 2023
@liangyepianzhou
[feat][txn]Implement transactionImpl (#984)
09dea66 
Master Issue:#932
### Motivation
Implement transaction coordinator client.
### Modifications
1. Implement transaction coordinator
2. implement transactionImpl
3. Implement transaction in producer and consumer API
@d80tb7
Copy link

d80tb7 commented Apr 4, 2023

Hi @liangyepianzhou good work on this so far! Am I right in thinking that transactions aren't available in the public api in 0.10? I can see there's an EnableTransaction in the client options, but as far as I can tell we can't yet send or receive messages with transactions?

@liangyepianzhou liangyepianzhou mentioned this issue Apr 5, 2023
Merged
1 task
@geniusjoe
Copy link
Contributor

Since merged in 953, can we close this issue as completed? @RobertIndie

@crossoverJie
Copy link
Member

@geniusjoe Done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@liangyepianzhou liangyepianzhou

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@d80tb7 @crossoverJie @geniusjoe @liangyepianzhou