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

move irrelevant code out of client-java repo #519

Closed
zz-jason opened this issue Feb 15, 2022 · 4 comments · Fixed by #563
Closed

move irrelevant code out of client-java repo #519

zz-jason opened this issue Feb 15, 2022 · 4 comments · Fixed by #563
Labels
type/enhancement enhancement other than feature or bugfix

Comments

@zz-jason
Copy link
Member

zz-jason commented Feb 15, 2022

Feature Request

Is your feature request related to a problem? Please describe:

client-java is a SDK for interaction between java apps and tikv cluster, just like client-go, client-rust

there are some packages that are irrelevant to the target goals, like:

  1. org.tikv.cdc: a library to manipulate CDC events
  2. org.tikv.br: a library to manipulate BR SST files
  3. coprocessor

Describe the feature you'd like:

move these irrelevant libraries to a better place, for example, CDC an be can be moved to https://github.com/tikv/migration

Describe alternatives you've considered:

N/A

Teachability, Documentation, Adoption, Migration Strategy:

N/A

@zz-jason zz-jason added type/feature-request New feature request type/enhancement enhancement other than feature or bugfix and removed type/feature-request New feature request labels Feb 15, 2022
@zz-jason
Copy link
Member Author

@birdstorm @marsishandsome PTAL

@birdstorm
Copy link
Collaborator

Agree. The support for CDC and BR could be moved to an individual library.

@marsishandsome
Copy link
Collaborator

LGTM

@github-actions
Copy link

This issue is stale because it has been open 30 days with no activity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement enhancement other than feature or bugfix
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants