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

MinMax Index Supports Nullable DataType. #4787

Closed
hehechen opened this issue Apr 28, 2022 · 1 comment · Fixed by #4792 or #5153
Closed

MinMax Index Supports Nullable DataType. #4787

hehechen opened this issue Apr 28, 2022 · 1 comment · Fixed by #4792 or #5153
Assignees
@hehechen
Labels
type/enhancement The issue or PR belongs to an enhancement.

Comments

@hehechen
Copy link
Contributor

Enhancement

Currently, MinMax Index does not support Nullable datatype. We can extract the nested data type from DataTypeNullable and reuse existing MinMax index code.
@hehechen hehechen added the type/enhancement The issue or PR belongs to an enhancement. label Apr 28, 2022
@hehechen hehechen self-assigned this Apr 28, 2022
@hehechen hehechen mentioned this issue Apr 28, 2022
Merged
12 tasks
ti-chi-bot pushed a commit that referenced this issue May 10, 2022
@hehechen
MinMax Index Supports Nullable DataType (#4792)
0e2b7f7 
close #4787
@hehechen hehechen mentioned this issue May 16, 2022
Merged
12 tasks
@JaySon-Huang JaySon-Huang reopened this May 16, 2022
@JaySon-Huang
Copy link
Contributor

Reopen since we don't plan to release it in 6.1 version. Will do later.

ti-chi-bot pushed a commit that referenced this issue May 16, 2022
@hehechen
Revert "MinMax Index Supports Nullable DataType" (#4895)
07f39f9 
ref #4787
@JaySon-Huang JaySon-Huang mentioned this issue May 25, 2022
Closed
@hehechen hehechen mentioned this issue Jun 15, 2022
Merged
12 tasks
ti-chi-bot pushed a commit that referenced this issue Jun 20, 2022
@hehechen
MinMax Index Supports Nullable DataType (#5153)
604b0de 
close #4787
@hehechen hehechen mentioned this issue Jun 29, 2022
Merged
12 tasks
ti-chi-bot pushed a commit that referenced this issue Jul 15, 2022
@hehechen
Add UT for MinMax Index (#5254)
ec717d8 
ref #4787
@hongyunyan hongyunyan mentioned this issue Sep 13, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hehechen hehechen

Labels
type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Milestone
No milestone
2 participants
@JaySon-Huang @hehechen