Skip to content

Update multi-value-esql-controls.md#5689

Open
mjmbischoff wants to merge 2 commits intomainfrom
mjmbischoff-doc-patch-mv-controls
Open

Update multi-value-esql-controls.md#5689
mjmbischoff wants to merge 2 commits intomainfrom
mjmbischoff-doc-patch-mv-controls

Conversation

@mjmbischoff
Copy link
Copy Markdown
Contributor

Summary

Param order addressed(otherwise docs with null value in field are always returned), adding reference to MV_INTERSECTS for any vs all semantics.

Generative AI disclosure

  1. Did you use a generative AI (GenAI) tool to assist in creating this contribution?
  • Yes
  • No

Param order addressed, adding reference to MV_INTERSECTS for any vs all semantics.
@mjmbischoff mjmbischoff requested review from a team as code owners March 31, 2026 08:42
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 31, 2026

🔍 Preview links for changed docs

⏳ Building and deploying preview... View progress

This comment will be updated with preview links when the build is complete.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 31, 2026

✅ Vale Linting Results

No issues found on modified lines!


The Vale linter checks documentation changes against the Elastic Docs style guide.

To use Vale locally or report issues, refer to Elastic style guide for Vale.

@mjmbischoff
Copy link
Copy Markdown
Contributor Author

#5684 already addresses the param order.

Copy link
Copy Markdown
Member

@leemthompo leemthompo left a comment

Choose a reason for hiding this comment

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

I have one language suggestion :)


2. When defining the control, select the **Allow multiple selections** option.
:::{note}
[`MV_CONTAINS`](elasticsearch://reference/query-languages/esql/functions-operators/mv-functions/mv_contains.md) will check if _all_ values are present, use [`MV_INTERSECTS`](elasticsearch://reference/query-languages/esql/functions-operators/mv-functions/mv_intersects.md) if you want _any_ present semantics.
Copy link
Copy Markdown
Member

@leemthompo leemthompo Mar 31, 2026

Choose a reason for hiding this comment

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

Suggested change
[`MV_CONTAINS`](elasticsearch://reference/query-languages/esql/functions-operators/mv-functions/mv_contains.md) will check if _all_ values are present, use [`MV_INTERSECTS`](elasticsearch://reference/query-languages/esql/functions-operators/mv-functions/mv_intersects.md) if you want _any_ present semantics.
[`MV_CONTAINS`](elasticsearch://reference/query-languages/esql/functions-operators/mv-functions/mv_contains.md) checks that _all_ subset values are present. Use [`MV_INTERSECTS`](elasticsearch://reference/query-languages/esql/functions-operators/mv-functions/mv_intersects.md) instead if matching _any_ subset value is enough.

minor suggestion to tighten up language—it's not perfect just felt the final sentence was a bit jargony and could be more concrete :)

@florent-leborgne
Copy link
Copy Markdown
Member

Hey, @mjmbischoff, thanks for the addition there. Since this section is explicitly about how to enable variable controls that can accept multiple values, are we saying that MV_INTERSECTS now also allows this? My initial understanding was that only MV_CONTAINS allowed this (unless there has been some recent development I haven't caught up on yet). CC @stratoula can you confirm?


2. When defining the control, select the **Allow multiple selections** option.
:::{note}
[`MV_CONTAINS`](elasticsearch://reference/query-languages/esql/functions-operators/mv-functions/mv_contains.md) will check if _all_ values are present, use [`MV_INTERSECTS`](elasticsearch://reference/query-languages/esql/functions-operators/mv-functions/mv_intersects.md) if you want _any_ present semantics.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hey! Can you explain to me which is the difference between MV_CONTAINS with the MV_INTERSECTS we want to add here?

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.

4 participants