Skip to content

Conversation

@Darkheir
Copy link
Contributor

Description

Add support for prefix and wildcard query operators in the elasticsearch DSL

How was this PR tested?

Added unit tests

Signed-off-by: Darkheir <raphael.cohen@sekoia.io>
@rdettai-sk
Copy link
Collaborator

Could you please add a test in quickwit/rest-api-tests/scenarii/es_compatibility that you also run against ES? thanks!

Signed-off-by: Darkheir <raphael.cohen@sekoia.io>
Signed-off-by: Darkheir <raphael.cohen@sekoia.io>
@Darkheir Darkheir force-pushed the feat/elastic_dsl_prefix_wildcard branch from bf5497e to 4c019ee Compare November 28, 2025 14:04
Signed-off-by: Darkheir <raphael.cohen@sekoia.io>
@Darkheir Darkheir force-pushed the feat/elastic_dsl_prefix_wildcard branch from 0da0f62 to f2681ad Compare November 28, 2025 15:04
@Darkheir
Copy link
Contributor Author

Darkheir commented Dec 2, 2025

Could you please add a test in quickwit/rest-api-tests/scenarii/es_compatibility that you also run against ES? thanks!

Done :-)

Copy link
Contributor

@trinity-1686a trinity-1686a left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm. Can you just change escape sequences to raw strings before we merge?

you don't need to add boost (though you're free to, here or in a follow up, if you want)

Signed-off-by: Darkheir <raphael.cohen@sekoia.io>
@Darkheir
Copy link
Contributor Author

Darkheir commented Dec 2, 2025

Thank you for the review :-)

I just applied your suggestions.

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.

3 participants