Skip to content
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

[8.x] Add selector syntax to index expressions (#118614) #119832

Merged
merged 12 commits into from
Jan 10, 2025

Conversation

jbaiera
Copy link
Member

@jbaiera jbaiera commented Jan 9, 2025

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

This PR introduces a new syntactical feature to index expression resolution: The selector.

Selectors, denoted with a :: followed by a recognized suffix will allow users to specify which component of
an index abstraction they would like to operate on within an API call. In this case, an index abstraction is a
concrete index, data stream, or alias; Any abstraction that can be resolved to a set of indices/shards. We
define a component of an index abstraction to be some searchable unit of the index abstraction.

(cherry picked from commit c3839e1)

# Conflicts:
#	modules/data-streams/src/internalClusterTest/java/org/elasticsearch/datastreams/IngestFailureStoreMetricsIT.java
#	server/src/main/java/org/elasticsearch/TransportVersions.java
#	server/src/test/java/org/elasticsearch/action/OriginalIndicesTests.java
Copy link
Contributor

github-actions bot commented Jan 9, 2025

Documentation preview:

@gmarouli
Copy link
Contributor

gmarouli commented Jan 9, 2025

We should cherry-pick #119875 here too.

@gmarouli
Copy link
Contributor

Hey @jbaiera , I have been keeping this up to date with the test fixes we have so far.

@jbaiera jbaiera merged commit dcb1394 into elastic:8.x Jan 10, 2025
15 checks passed
@jbaiera jbaiera deleted the backport/8.x/pr-118614 branch January 10, 2025 22:39
tlrx added a commit to tlrx/elasticsearch that referenced this pull request Jan 14, 2025
Mixed cluster tests failed because a serialization change wasn't backported during a couple of days (elastic#119832).

Now the change is backported, we can reenable those tests.

Closes elastic#119990
Closes elastic#119989
Closes elastic#119980
Closes elastic#119979
Closes elastic#119978
tlrx added a commit that referenced this pull request Jan 14, 2025
#120113)

Mixed cluster tests failed because a serialization change wasn't backported during a couple of days (#119832).

Now the change is backported, we can reenable those tests.

Closes #119990
Closes #119989
Closes #119980
Closes #119979
Closes #119978
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants