Skip to content

Conversation

kkrik-es
Copy link
Contributor

Backports the following changes to branch 8.x:

* Introduce mode `subobjects=auto` for objects

* Update docs/changelog/110524.yaml

* compilation error

* tests and fixes

* refactor

* spotless

* more tests

* fix nested objects

* fix test

* update fetch test

* add QA coverage

* update tests

* update tests

* update tests

* Apply auto-flattening to `subobjects: auto`

* Update docs/changelog/112092.yaml

* sync

* dont flatten subobjects auto

* refine test

* fix path for nested flattened objects and dynamic

* document `subobjects: auto`

* Apply suggestions from code review

Co-authored-by: Felix Barnsteiner <[email protected]>

* comment updates

* restore indentation in comment

* update comment

* update comment

* update comment

* update comment

* rename isFlattenable

* add test for dynamic template

* fix copy_to and noop dynamic updates

* tests

* update comment

* fix tests

* update cluster feature in yaml test

* address comments

---------

Co-authored-by: Felix Barnsteiner <[email protected]>
(cherry picked from commit fffe884)

# Conflicts:
#	modules/dot-prefix-validation/build.gradle
#	rest-api-spec/build.gradle
@kkrik-es kkrik-es added >enhancement backport auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) Team:StorageEngine :StorageEngine/Mapping The storage related side of mappings v8.16.0 labels Sep 26, 2024
@kkrik-es kkrik-es self-assigned this Sep 26, 2024
Copy link
Contributor

Documentation preview:

@kkrik-es kkrik-es marked this pull request as ready for review September 26, 2024 09:00
@kkrik-es kkrik-es requested a review from a team as a code owner September 26, 2024 09:00
@elasticsearchmachine elasticsearchmachine merged commit 8539876 into elastic:8.x Sep 26, 2024
16 checks passed
@kkrik-es kkrik-es deleted the backport/8.x/pr-112092 branch September 26, 2024 10:17
elasticsearchmachine pushed a commit that referenced this pull request Sep 26, 2024
The skip test entries were added in
#113584, no longer needed
after backporting it.
dnhatn pushed a commit to dnhatn/elasticsearch that referenced this pull request Sep 30, 2024
* Apply auto-flattening to `subobjects: auto` (elastic#112092)

* Introduce mode `subobjects=auto` for objects

* Update docs/changelog/110524.yaml

* compilation error

* tests and fixes

* refactor

* spotless

* more tests

* fix nested objects

* fix test

* update fetch test

* add QA coverage

* update tests

* update tests

* update tests

* Apply auto-flattening to `subobjects: auto`

* Update docs/changelog/112092.yaml

* sync

* dont flatten subobjects auto

* refine test

* fix path for nested flattened objects and dynamic

* document `subobjects: auto`

* Apply suggestions from code review

Co-authored-by: Felix Barnsteiner <[email protected]>

* comment updates

* restore indentation in comment

* update comment

* update comment

* update comment

* update comment

* rename isFlattenable

* add test for dynamic template

* fix copy_to and noop dynamic updates

* tests

* update comment

* fix tests

* update cluster feature in yaml test

* address comments

---------

Co-authored-by: Felix Barnsteiner <[email protected]>
(cherry picked from commit fffe884)

# Conflicts:
#	modules/dot-prefix-validation/build.gradle
#	rest-api-spec/build.gradle

* Update build.gradle
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >enhancement :StorageEngine/Mapping The storage related side of mappings Team:StorageEngine v8.16.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants