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

Valid query considered invalid by the editor #77

Open
fmassot opened this issue Feb 19, 2024 · 3 comments
Open

Valid query considered invalid by the editor #77

fmassot opened this issue Feb 19, 2024 · 3 comments
Assignees

Comments

@fmassot
Copy link
Contributor

fmassot commented Feb 19, 2024

image

@ddelemeny
Copy link
Contributor

This problem comes from the 3rd party parser we use to power the linter on the frontend. Its grammar doesn't support / in field names.

It does not impede the ability to execute the query, query runs are separated from linting.

Long term problem is that this dependency is unmaintained and relies on an unmaintained parser generator itself. We should probably maintain our own parser if there's no better option out there.

@trinity-1686a
Copy link

I don't know if that's actually a good idea, but tantivy query parser is pretty self contained, and shouldn't be too hard to integrate as a wasm module in a js app

@quanghungb
Copy link

I have the same issue here with the 'IN' https://quickwit.io/docs/reference/query-language#term-set-fieldin-a-b-c
image

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

4 participants