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

Unify the "split client" package for BR and lightning #51533

Open
lance6716 opened this issue Mar 6, 2024 · 2 comments
Open

Unify the "split client" package for BR and lightning #51533

lance6716 opened this issue Mar 6, 2024 · 2 comments
Labels
type/enhancement The issue or PR belongs to an enhancement.

Comments

@lance6716
Copy link
Contributor

lance6716 commented Mar 6, 2024

Enhancement

now BR and lightning have separate functions to do one thing. Better to unify them

And the final goal is that lightning can use BR's new split&scatter strategy, which is first split to about #store, scatter them, and perform rest split to satisfy total region count

@lance6716 lance6716 added the type/enhancement The issue or PR belongs to an enhancement. label Mar 6, 2024
ti-chi-bot bot pushed a commit that referenced this issue Apr 10, 2024
3AceShowHand pushed a commit to 3AceShowHand/tidb that referenced this issue Apr 16, 2024
@okJiang
Copy link
Member

okJiang commented Jun 17, 2024

Can you tell me which modules still need to be unified for this issue? @lance6716

@lance6716
Copy link
Contributor Author

Can you tell me which modules still need to be unified for this issue? @lance6716

Do you mean why this issue is not finished? Reasons:

  1. Currently we don't have enough test clsuter to verify it
  2. I'm considering if the "split client" should have built-in coarse-grained + finer-grained two level split, don't let caller bother it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Development

No branches or pull requests

2 participants