quickwit-oss / quickwit-datasource

Quickwit data source for Grafana
GNU Affero General Public License v3.0
44 stars 10 forks source link

Valid query considered invalid by the editor #77

Open fmassot opened 8 months ago

fmassot commented 8 months ago

image

ddelemeny commented 8 months ago

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 commented 8 months ago

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 commented 4 months ago

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