Skip to content

Conversation

jbaiera
Copy link
Member

@jbaiera jbaiera commented Sep 17, 2025

Adds the new field access pattern to the specification. I didn't update any example code because this is not really expected to be a common option to set.

Relates to:

Closes #5310

@jbaiera jbaiera added specification skip-backport This pull request should not be backported labels Sep 17, 2025
@jbaiera jbaiera requested a review from pquentin September 17, 2025 22:31
@pquentin pquentin mentioned this pull request Sep 18, 2025
Copy link
Member

@pquentin pquentin left a comment

Choose a reason for hiding this comment

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

Thanks! I opened #5310 to get CI results, but next time please open this from a branch instead of a fork. I realize that this is an annoying limitation, hopefully it will go away at some point.

@jbaiera
Copy link
Member Author

jbaiera commented Sep 18, 2025

but next time please open this from a branch instead of a fork

Ah, my apologies! Didn't realize that was a restriction. Will do in the future. Would you like me to reopen it as a branch?

Copy link
Member

@pquentin pquentin left a comment

Choose a reason for hiding this comment

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

Thanks! LGTM.

No need to open from a branch as no new changes are planned.

@pquentin pquentin merged commit 34dfe8b into elastic:main Sep 22, 2025
19 of 21 checks passed
@jbaiera jbaiera deleted the add-ingest-field-access-pattern branch September 22, 2025 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-backport This pull request should not be backported specification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants