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

0.7.0 Searcher web UI fails to parse multi-line queries #4408

Closed
to266 opened this issue Jan 17, 2024 · 3 comments · Fixed by quickwit-oss/tantivy#2302 or #4409
Closed

0.7.0 Searcher web UI fails to parse multi-line queries #4408

to266 opened this issue Jan 17, 2024 · 3 comments · Fixed by quickwit-oss/tantivy#2302 or #4409
Assignees
Labels
bug Something isn't working

Comments

@to266
Copy link

to266 commented Jan 17, 2024

Describe the bug
Searcher web UI fails to parse multi-line queries

Steps to reproduce (if applicable)
Steps to reproduce the behavior:

  1. Deploy to k8s via https://github.com/quickwit-oss/helm-charts/releases/tag/quickwit-0.5.3
  2. Start feeding otel logs
  3. Attempt to query
ERROR 
AND resource_attributes.k8s.deployment.name:something
  1. Get error
Error: { "message": "failed to parse query: `ERROR \nAND resource_attributes.k8s.deployment.name:something`" }

Expected behavior
With 0.6.4 it used to work seamlessly

Configuration:
Please provide:

  1. Output of quickwit --version
Quickwit v0.7.0 (f218294 2024-01-05T02:11:51Z
  1. The index_config.yaml
    Actually the default whatever quickwit creates. Even with 0.7.0 it still creates otel-logs-v0_6, despite the release notes
@to266 to266 added the bug Something isn't working label Jan 17, 2024
@trinity-1686a
Copy link
Contributor

the parser was rewritten from using combine to using nom. It seems what they call space differs. nom only consider and \t to be space, combine allows \n. With nom, we should use multispace instead to match \n

@trinity-1686a
Copy link
Contributor

trinity-1686a commented Jan 17, 2024

not actually fixed until we pull the new commit from tantivy into quickwit

@guilload
Copy link
Member

@to266, FYI, we're planning on releasing Quickwit 0.7.1 this week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
3 participants