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

IndexMerge enhancement list #31699

Open
guo-shaoge opened this issue Jan 14, 2022 · 1 comment
Open

IndexMerge enhancement list #31699

guo-shaoge opened this issue Jan 14, 2022 · 1 comment
Labels
type/enhancement The issue or PR belongs to an enhancement.

Comments

@guo-shaoge
Copy link
Collaborator

guo-shaoge commented Jan 14, 2022 •
edited
Loading

Enhancement

  1. CNF support.
  2. Support IndexMerge used in cache table/temporary table. (Maybe dup with small cached table doesn't support index merge #29682).
  3. Support PointGet/BatchPointGet used in IndexMerge for better performance. Check detailed plan in comment.
  4. Better row est for IndexMerge when there are exprs cannot be pushed to TiKV. So user no need to use hint to use IndexMerge.
  5. Better code. Maybe we can merge IndexMergeReaderExecutor and IndexLookUpReaderExecutor into one. Because their code is similar and we can reduce duplicated code.
@guo-shaoge guo-shaoge added the type/enhancement The issue or PR belongs to an enhancement. label Jan 14, 2022
@guo-shaoge
Copy link
Collaborator Author

For 3, we can support plan like, we only support IndexReader or TableReader as subreader for IndexMerge. PointGet may be more efficient in some case.

IndexMergeReader
    |- PointGet
    |- TableReader

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

No branches or pull requests
1 participant
@guo-shaoge