Skip to content

[Transaction] Support produce batch transaction messages #8011

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
Closed
gaoran10 wants to merge 2 commits into from
Closed

[Transaction] Support produce batch transaction messages #8011

gaoran10 wants to merge 2 commits into from

Conversation

@gaoran10
Copy link
Contributor

@gaoran10 gaoran10 commented Sep 8, 2020

Motivation

Currently, we couldn't produce batch transaction messages.

Modifications

Support produces batch transaction messages based on BatchMessageKeyBasedContainer.

Verifying this change

This change added tests and can be verified as follows:

  • org.apache.pulsar.client.transaction.EndToEndTest#batchProduceCommitTest

Does this pull request potentially affect one of the following parts:

If yes was chosen, please highlight the changes

  • Dependencies (does it add or upgrade a dependency): (no)
  • The public API: (no)
  • The schema: (no)
  • The default values of configurations: (no)
  • The wire protocol: (no)
  • The rest endpoints: (no)
  • The admin cli options: (no)
  • Anything that affects deployment: (no)

Documentation

  • Does this pull request introduce a new feature? (no)

@gaoran10 gaoran10 mentioned this pull request Sep 10, 2020
Closed
16 tasks
@codelipenghui codelipenghui added this to the 2.7.0 milestone Sep 11, 2020
@jiazhai
Copy link
Member

jiazhai commented Sep 15, 2020

todo: extends keybasedcontainer

@gaoran10 gaoran10 force-pushed the txn-produce-batch-message branch 2 times, most recently from 28fbdb4 to d5fcb4c Compare September 20, 2020 17:54
@gaoran10 gaoran10 force-pushed the txn-produce-batch-message branch from d5fcb4c to fd4a4a0 Compare September 25, 2020 03:03
@gaoran10 gaoran10 force-pushed the txn-produce-batch-message branch from fd4a4a0 to 2a2c6ef Compare October 15, 2020 06:43
@codelipenghui
Copy link
Contributor

codelipenghui commented Oct 31, 2020

Dup with #8415

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

Reviewers

No reviews

Assignees

@gaoran10 gaoran10

Labels

area/transaction

Projects

None yet

Milestone

2.7.0

Development

Successfully merging this pull request may close these issues.

3 participants

@gaoran10 @jiazhai @codelipenghui