-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
[console] autocomplete query dsl gaps #19318
[console] autocomplete query dsl gaps #19318
Conversation
💚 Build Succeeded |
percolate: { | ||
field: '', | ||
document: {} | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think more parameters need to be supported? https://www.elastic.co/guide/en/elasticsearch/reference/current/query-dsl-percolate-query.html#_parameters_10
|
||
const SPAN_MULTI_QUERIES = { | ||
wildcard: { | ||
__scope_link: '.wildcard' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ycombinator I've addressed your feedback. Good catch on the inner template thing -- that never worked but good to get it working. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
💔 Build Failed |
💔 Build Failed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah nice, I like what you did with extracting the template definitions. LGTM!
* adding wrapper query DSL support * adding span_multi query DSL support for autocomplete * adding field_masking_span query DSL support to autocomplete * adding percolator query DSL support to autcomplete * fixing templates with nested query types * adding missing percolate params
* adding wrapper query DSL support * adding span_multi query DSL support for autocomplete * adding field_masking_span query DSL support to autocomplete * adding percolator query DSL support to autcomplete * fixing templates with nested query types * adding missing percolate params
This adds 4 query types to autocomplete in console. Closes #19178