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

[Transaction coordinator]Add transaction metadata store service #5504

Merged
merged 4 commits into from
Nov 6, 2019
Merged

[Transaction coordinator]Add transaction metadata store service #5504

merged 4 commits into from
Nov 6, 2019

Conversation

codelipenghui
Copy link
Contributor

Motivation

Add transaction metadata store service and auto create 1 transaction metadata store with
id 0

Verifying this change

Add new unit tests for transaction metadata store service

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): (yes)
  • 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? (yes)

This is a internal mechanism for transaction streaming

@codelipenghui
Copy link
Contributor Author

@congbobo184 please help review this changes

@codelipenghui
Copy link
Contributor Author

run java8 tests

@codelipenghui codelipenghui self-assigned this Oct 30, 2019
@codelipenghui codelipenghui added this to the 2.5.0 milestone Oct 30, 2019
@codelipenghui
Copy link
Contributor Author

run java8 tests
run integration tests

2 similar comments
@codelipenghui
Copy link
Contributor Author

run java8 tests
run integration tests

@codelipenghui
Copy link
Contributor Author

run java8 tests
run integration tests

@codelipenghui
Copy link
Contributor Author

run integration tests

@sijie
Copy link
Member

sijie commented Oct 31, 2019

run integration tests

3 similar comments
@codelipenghui
Copy link
Contributor Author

run integration tests

@codelipenghui
Copy link
Contributor Author

run integration tests

@codelipenghui
Copy link
Contributor Author

run integration tests

@sijie
Copy link
Member

sijie commented Nov 2, 2019

@codelipenghui can you rebase this pull request?

@codelipenghui
Copy link
Contributor Author

run integration tests

6 similar comments
@sijie
Copy link
Member

sijie commented Nov 3, 2019

run integration tests

@aahmed-se
Copy link
Contributor

run integration tests

@codelipenghui
Copy link
Contributor Author

run integration tests

@codelipenghui
Copy link
Contributor Author

run integration tests

@codelipenghui
Copy link
Contributor Author

run integration tests

@sijie
Copy link
Member

sijie commented Nov 4, 2019

run integration tests

@sijie
Copy link
Member

sijie commented Nov 5, 2019

run integration tests

@congbobo184
Copy link
Contributor

run Integration Tests

@codelipenghui
Copy link
Contributor Author

run java8 tests

1 similar comment
@codelipenghui
Copy link
Contributor Author

run java8 tests

@codelipenghui codelipenghui merged commit e0b36f9 into apache:master Nov 6, 2019
@codelipenghui codelipenghui deleted the tc_startup branch May 19, 2021 05:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sijie sijie sijie approved these changes

@jiazhai jiazhai Awaiting requested review from jiazhai

Assignees

@codelipenghui codelipenghui

Labels
area/broker area/transaction
Projects
None yet
Milestone
2.5.0
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@codelipenghui @sijie @aahmed-se @congbobo184 @tisonkun