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

Exists query does not work #2508

Closed
Zhincore opened this issue Oct 7, 2024 · 3 comments
Closed

Exists query does not work #2508

Zhincore opened this issue Oct 7, 2024 · 3 comments

Comments

@Zhincore
Copy link

Zhincore commented Oct 7, 2024

Describe the bug

Which version of tantivy are you using?
0.22.0

To Reproduce

  1. Have a Tantivy setup, I didn't find any correlation with field types or anything.
  2. Have QueryParser, e.g. let query_parser = QueryParser::for_index(&index, vec![ ...your default fields... ])
  3. Try to parse an exists query query_parser.parse_query("name:*")
  4. You get the above error :(
@LebranceBW
Copy link
Contributor

It's odd that ExistsQuery has been implemented, but this particular query pattern does not utilize it.

If there's no specific reason behind it, I'd be happy to hook them up.

@fulmicoton
Copy link
Collaborator

fulmicoton commented Oct 9, 2024

ExistsQuery work. They are not integrated with the query parser. This is not a bug as long as documentation says otherwise.

Closing this issue.

@LebranceBW
It has been implemented for Quickwit, which uses its own query parser.
Generally speaking I strongly recommend people building something serious to implement their own query parser.
Feel free to open a feature ticket if you want to contribute the addition of this syntax in tantivy's query parser.

@LebranceBW
Copy link
Contributor

ExistsQuery work. They are not integrated with the query parser. This is not a bug as long as documentation says otherwise.

Closing this issue.

@LebranceBW It has been implemented for Quickwit, which uses its own query parser. Generally speaking I strongly recommend people building something serious to implement their own query parser. Feel free to open a feature ticket if you want to contribute the addition of this syntax in tantivy's query parser.

I've submitted issue #2514 and PR #2513 for the newly supported syntax. Could you please take a moment to review them? @fulmicoton

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants