[ES|QL] Add Documentation for MMR command#143809
Conversation
🔍 Preview links for changed docs |
ℹ️ Important: Docs version tagging👋 Thanks for updating the docs! Just a friendly reminder that our docs are now cumulative. This means all 9.x versions are documented on the same page and published off of the main branch, instead of creating separate pages for each minor version. We use applies_to tags to mark version-specific features and changes. Expand for a quick overviewWhen to use applies_to tags:✅ At the page level to indicate which products/deployments the content applies to (mandatory) What NOT to do:❌ Don't remove or replace information that applies to an older version 🤔 Need help?
|
carlosdelest
left a comment
There was a problem hiding this comment.
LGTM - please double check the applies_to versions, as they look wrong
docs/reference/query-languages/esql/_snippets/commands/layout/mmr.md
Outdated
Show resolved
Hide resolved
leemthompo
left a comment
There was a problem hiding this comment.
Docs wise: one question and one ++ :)
docs/reference/query-languages/esql/_snippets/commands/layout/mmr.md
Outdated
Show resolved
Hide resolved
docs/reference/query-languages/esql/_snippets/commands/layout/mmr.md
Outdated
Show resolved
Hide resolved
| - file: esql/commands/keep.md | ||
| - file: esql/commands/limit.md | ||
| - file: esql/commands/lookup-join.md | ||
| - file: esql/commands/mmr.md |
There was a problem hiding this comment.
FYI we shouldn't merge this until the command is live on serverless
If you want to merge in hidden mode, you can do the following, and publish at a later date
| - file: esql/commands/mmr.md | |
| - hidden: esql/commands/mmr.md |
There was a problem hiding this comment.
FYI this PR is now merged so the command should be available.
ioanatia
left a comment
There was a problem hiding this comment.
Before we merge this PR, we need to make sure that MMR is available outside of release builds - we need to regen the syntax:
and also make sure this capability is available in release builds:
also let's make sure we remove this capability before we make this available in release builds:
We only need MMR_V2 when we release.
Thanks for calling this out - I'll open up a new PR today for this, then circle back to these docs |
PR created to pull this out of dev/snapshot builds: #143867 |
|
Pinging @elastic/es-search-relevance (Team:Search Relevance) |
docs/reference/query-languages/esql/_snippets/commands/layout/mmr.md
Outdated
Show resolved
Hide resolved
docs/reference/query-languages/esql/_snippets/commands/layout/mmr.md
Outdated
Show resolved
Hide resolved
docs/reference/query-languages/esql/_snippets/commands/layout/mmr.md
Outdated
Show resolved
Hide resolved
|
pinging @ioanatia for review |
leemthompo
left a comment
There was a problem hiding this comment.
One last version tag and LGTM
docs/reference/query-languages/esql/_snippets/commands/layout/mmr.md
Outdated
Show resolved
Hide resolved
docs/reference/query-languages/esql/_snippets/commands/layout/mmr.md
Outdated
Show resolved
Hide resolved
docs/reference/query-languages/esql/_snippets/commands/layout/mmr.md
Outdated
Show resolved
Hide resolved
* add docs for MMR command * remove 9.4 label for layout * doc updates * update docs; add command comments per Nik * update regen docs * doc cleanups
fixes error introduced in #143809
fixes error introduced in #143809
…145338) fixes error introduced in elastic#143809
…145338) fixes error introduced in elastic#143809
* add docs for MMR command * remove 9.4 label for layout * doc updates * update docs; add command comments per Nik * update regen docs * doc cleanups
…145338) fixes error introduced in elastic#143809
…145338) fixes error introduced in elastic#143809
Adds documentation for the ES|QL MMR command.