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

Using the filtering results of rough indexes to reduce the calculation of filters #9103

Closed
JinheLin opened this issue May 30, 2024 · 0 comments · Fixed by #9318
Closed

Using the filtering results of rough indexes to reduce the calculation of filters #9103

JinheLin opened this issue May 30, 2024 · 0 comments · Fixed by #9318
Assignees
@JinheLin
Labels
type/enhancement The issue or PR belongs to an enhancement.

Comments

@JinheLin
Copy link
Contributor

Enhancement
@JinheLin JinheLin added the type/enhancement The issue or PR belongs to an enhancement. label May 30, 2024
@JinheLin JinheLin self-assigned this May 30, 2024
@JinheLin JinheLin mentioned this issue May 30, 2024
Merged
12 tasks
@JinheLin JinheLin mentioned this issue Jun 13, 2024
Merged
12 tasks
ti-chi-bot bot added a commit that referenced this issue Jun 13, 2024
@JinheLin @JaySon-Huang @ti-chi-bot
Storages: Refine build filters in StorageDeltaMerge (#9104)
d399917 
ref #9103

Move some builder functions of filters to be a static member function of each class.

Co-authored-by: JaySon <tshent@qq.com>
Co-authored-by: JaySon-Huang <tshent@qq.com>
Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
@ti-chi-bot ti-chi-bot bot closed this as completed in 380a5f3 Jun 13, 2024
@JaySon-Huang JaySon-Huang reopened this Jun 13, 2024
@JinheLin JinheLin mentioned this issue Jun 17, 2024
Merged
12 tasks
ti-chi-bot bot pushed a commit that referenced this issue Jun 17, 2024
@JinheLin @JaySon-Huang
Storages: Add comments to MinMaxIndex (#9150)
c783e54 
ref #9103

Co-authored-by: JaySon <tshent@qq.com>
@JinheLin JinheLin mentioned this issue Jun 18, 2024
Merged
12 tasks
ti-chi-bot bot pushed a commit that referenced this issue Jun 18, 2024
@JinheLin @JaySon-Huang
Storages: Refine MinMaxIndex (#9151)
86e89f7 
ref #9103

Co-authored-by: JaySon <tshent@qq.com>
@JinheLin JinheLin mentioned this issue Jun 18, 2024
Merged
12 tasks
ti-chi-bot bot pushed a commit that referenced this issue Jun 19, 2024
@JinheLin @JaySon-Huang
Storages: MinMaxIndex::checkIsNull returns RSResult::All if a pack on…
7e70abd 
…ly contains null marks and delete marks (#9152)

ref #9103

Co-authored-by: JaySon-Huang <tshent@qq.com>
@JinheLin JinheLin mentioned this issue Jun 20, 2024
Closed
12 tasks
@JinheLin JinheLin mentioned this issue Jul 24, 2024
Merged
12 tasks
ti-chi-bot bot pushed a commit that referenced this issue Jul 25, 2024
@JinheLin @Lloyd-Pottiger
Storages: Fix statistical data of DMFilePackFilter (#9254)
a4f0ec6 
ref #9103

Co-authored-by: Lloyd-Pottiger <60744015+Lloyd-Pottiger@users.noreply.github.com>
@JinheLin JinheLin mentioned this issue Aug 7, 2024
Merged
12 tasks
ti-chi-bot bot added a commit that referenced this issue Aug 13, 2024
@JinheLin @JaySon-Huang @ti-chi-bot
Storages: Add whether has null in the result of MinMaxIndex (#9288)
b4e8599 
ref #9103

1. Refine `RSResult`, make it can express whether null value is contained.
2. In `MinMaxIndex`, check if the pack has null value, and if so, add information containing the null value to the returned result.

Co-authored-by: JaySon <tshent@qq.com>
Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
@JinheLin JinheLin mentioned this issue Aug 19, 2024
Merged
12 tasks
@ti-chi-bot ti-chi-bot bot closed this as completed in 2652edb Aug 22, 2024
@JaySon-Huang JaySon-Huang mentioned this issue Aug 26, 2024
Merged
12 tasks
This was referenced Sep 27, 2024
Closed
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JinheLin JinheLin

Labels
type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Storages: Skip filter if all data match in Min-Max index JinheLin/tiflash-1
2 participants
@JaySon-Huang @JinheLin