[DOCS] Reformat match phrase prefix query#45209
Conversation
jrodewig
added
>docs
|
Pinging @elastic/es-search |
|
Pinging @elastic/es-docs |
mayya-sharipova
left a comment
mayya-sharipova
left a comment
There was a problem hiding this comment.
@jrodewig Thanks, nice reorganization!
| [[match-phrase-prefix-query-ex-request]] | ||
| ==== Example request | ||
|
|
||
| The following search returns documents that phrases beginning with `quick brown |
There was a problem hiding this comment.
documents that phrases -> documents that contain phrases?
There was a problem hiding this comment.
Fixed with 8f391fa.
| ==== Notes | ||
|
|
||
| [[match-phrase-prefix-autocomplete]] | ||
| ===== Avoid using the match phrase prefix query for search autocompletion |
There was a problem hiding this comment.
I think "avoid" is a strong word here. I would rather say "Prefer not using". Because for some use cases (for a quick start etc), it is a valid use case.
There was a problem hiding this comment.
I've changed this heading to "Using..." with bb2e045. This will make the nuance of that use case a bit more clear and encourage users to read the block.
| While easy to set up, using the `match_phrase_prefix` query for search | ||
| autocompletion can sometimes produce confusing results. | ||
|
|
||
| For example, consider the query string `quick brown f`. This query works by |
There was a problem hiding this comment.
f`. This
brown). Then
nit: are there two spaces after .?
|
Thanks @mayya-sharipova. |
4 participants
Updates the
match_phrase_prefixquery to use the new query format.This creates separate sections for the example request, parameters, and notes.
This is part of #40977, an effort to standardize documentation for query types.
Preview
http://elasticsearch_45209.docs-preview.app.elstc.co/guide/en/elasticsearch/reference/master/query-dsl-match-query-phrase-prefix.html