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

Issue #18: add queryParser module #32

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Conversation

idrissneumann
Copy link
Collaborator

No description provided.

@idrissneumann
Copy link
Collaborator Author

First version, we'll see if we need to improve the parser using a lucene dependancy in a second version:

Screenshot 2023-11-30 at 17 04 32 Screenshot 2023-11-30 at 17 04 21

@fmassot
Copy link
Contributor

fmassot commented Dec 8, 2023

Let's not merge this as is. I will work on quickwit side to return the list of search keywords in the elasticsearch response so that we just have to add them to the dataframe.

@idrissneumann idrissneumann marked this pull request as draft December 8, 2023 15:35
@idrissneumann
Copy link
Collaborator Author

Let's not merge this as is. I will work on quickwit side to return the list of search keywords in the elasticsearch response so that we just have to add them to the dataframe.

I just put this pr in draft again, I'll rework it when the quickwit response will provide the keywords.

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

Successfully merging this pull request may close these issues.

2 participants