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

Enable str fast field based range queries. #2023

Closed
fulmicoton opened this issue May 3, 2023 · 2 comments · Fixed by #2453
Closed

Enable str fast field based range queries. #2023

fulmicoton opened this issue May 3, 2023 · 2 comments · Fixed by #2453
Assignees
@PSeitz
Labels
high priority

Comments

@fulmicoton
Copy link
Collaborator

fulmicoton commented May 3, 2023 •
edited
Loading

This is for quickwit. Which means we at least want a StrFastFieldRange{Query/Weight}
@fulmicoton fulmicoton changed the title Enable str based range queries. Enable str fast field based range queries. May 3, 2023
@fulmicoton
Copy link
Collaborator Author

@PSeitz Do we have this already I forgot?

@PSeitz
Copy link
Contributor

PSeitz commented Aug 15, 2023

Currently only u64 fast fields are supported.

If we can map the str ranges to term ids as a preprocessing, reusing the existing u64 range queries on fast fields should be straight forward.

@PSeitz PSeitz mentioned this issue Jul 12, 2024
Merged
PSeitz added a commit that referenced this issue Jul 15, 2024
@PSeitz
refactor fast field query
6cca1cd 
As preparation of #2023 and #1709

* Use Term to pass parameters
* merge u64 and ip fast field range query

Side note: I did not rename range_query_u64_fastfield, because then git can't track the changes.
PSeitz added a commit that referenced this issue Jul 15, 2024
@PSeitz
refactor fast field query (#2452)
1b40766 
As preparation of #2023 and #1709

* Use Term to pass parameters
* merge u64 and ip fast field range query

Side note: I did not rename range_query_u64_fastfield, because then git can't track the changes.
PSeitz added a commit that referenced this issue Jul 16, 2024
@PSeitz
add support for str fast field range query
fa51659 
Add support for range queries on fast fields, by converting term bounds to
term ordinals bounds.

closes #2023
@PSeitz PSeitz mentioned this issue Jul 16, 2024
Merged
PSeitz added a commit that referenced this issue Jul 16, 2024
@PSeitz
add support for str fast field range query
05bc5d1 
Add support for range queries on fast fields, by converting term bounds to
term ordinals bounds.

closes #2023
PSeitz added a commit that referenced this issue Jul 16, 2024
@PSeitz
add support for str fast field range query
792d645 
Add support for range queries on fast fields, by converting term bounds to
term ordinals bounds.

closes #2023
PSeitz added a commit that referenced this issue Jul 16, 2024
@PSeitz
add support for str fast field range query
8b58b67 
Add support for range queries on fast fields, by converting term bounds to
term ordinals bounds.

closes #2023
@PSeitz PSeitz closed this as completed in 7ebcc15 Jul 17, 2024
philippemnoel pushed a commit to paradedb/tantivy that referenced this issue Aug 31, 2024
@PSeitz @philippemnoel
refactor fast field query (quickwit-oss#2452)
d6e8ed7 
As preparation of quickwit-oss#2023 and quickwit-oss#1709

* Use Term to pass parameters
* merge u64 and ip fast field range query

Side note: I did not rename range_query_u64_fastfield, because then git can't track the changes.
philippemnoel pushed a commit to paradedb/tantivy that referenced this issue Aug 31, 2024
@PSeitz @philippemnoel
add support for str fast field range query (quickwit-oss#2453)
bec1f60 
* add support for str fast field range query

Add support for range queries on fast fields, by converting term bounds to
term ordinals bounds.

closes quickwit-oss#2023

* extend tests, rename

* update comment

* update comment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@PSeitz PSeitz

Labels
high priority
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

add support for str fast field range query quickwit-oss/tantivy
2 participants
@fulmicoton @PSeitz