-
Notifications
You must be signed in to change notification settings - Fork 25.6k
SQL: MATCH and QUERY documentation; one list of functions #40494
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
Conversation
Polish the functions pages and add a list of functions to the main Functions & Operators page.
|
Pinging @elastic/es-search |
costin
left a comment
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.
Excellent work!
My only comment would be to add a comment to LIKE and RLIKE section mentioning the advantages of full-text queries: faster, more powerful and accurate.
| include-tagged::{sql-specs}/docs/docs.csv-spec[simpleMatch] | ||
| ---- | ||
|
|
||
| However, it can also receive a list of fields and their corresponding optional `boost` value. In this case, {es-sql} will use a |
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 would add a basic explanation on boosting does in this example for SQL folks that might be familiar with it.
| additional configuration parameters (separated by semicolon `;`) for either `match` or `multi_match` queries. For example: | ||
|
|
||
| ["source","sql",subs="attributes,callouts,macros"] | ||
| ---- |
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.
Same here - explain what the example tries to achieve.
|
|
||
| {es-sql} provides a comprehensive set of built-in operators and functions: | ||
|
|
||
| * <<sql-operators, Operators>> |
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.
👍
matriv
left a comment
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. Great work here!
* Document MATCH and QUERY function predicates. * Polish the functions pages and add a list of functions to the main Functions & Operators page. (cherry picked from commit 4cec0ae)
* Document MATCH and QUERY function predicates. * Polish the functions pages and add a list of functions to the main Functions & Operators page. (cherry picked from commit 4cec0ae)
* Document MATCH and QUERY function predicates. * Polish the functions pages and add a list of functions to the main Functions & Operators page. (cherry picked from commit 4cec0ae)
This PR adds documentation for MATCH and QUERY function predicates, updates the Functions & Operators page to include the list of all available functions and polishes (&rearranges) some of the other functions pages.
Fixes #35076.
Fixes #36885.