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

Added known issue entry for synthetic source bug. #113269

Merged
merged 5 commits into from
Sep 23, 2024

Conversation

martijnvg
Copy link
Member

The bug was fixed as part of #112713

Copy link
Contributor

Documentation preview:

@martijnvg martijnvg marked this pull request as ready for review September 20, 2024 14:11
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-storage-engine (Team:StorageEngine)

docs/reference/release-notes/8.15.0.asciidoc Outdated Show resolved Hide resolved
If you use synthetic source then you may be affected by this bug if the following it true:
** If you have more fields then the `index.mapping.total_fields.limit` setting allows.
** If you use dynamic mappings and the `index.mapping.total_fields.ignore_dynamic_beyond_limit` setting is enabled.
** If you use `nested` field types in your mappings.
Copy link
Contributor

Choose a reason for hiding this comment

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

I am not sure why nested is mentioned. AFAIK it's not related?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, it is removed. Nested or arrays have nothing to doing with this. It is about when index.mapping.total_fields.limit is exceeded or index.mapping.total_fields.ignore_dynamic_beyond_limit is configured.

docs/reference/release-notes/8.15.1.asciidoc Outdated Show resolved Hide resolved
martijnvg and others added 4 commits September 20, 2024 19:07
Co-authored-by: Oleksandr Kolomiiets <[email protected]>
Co-authored-by: Oleksandr Kolomiiets <[email protected]>
@martijnvg martijnvg requested a review from lkts September 20, 2024 17:12
@martijnvg martijnvg merged commit c21342a into elastic:main Sep 23, 2024
5 checks passed
martijnvg added a commit to martijnvg/elasticsearch that referenced this pull request Sep 23, 2024
Added known issue entry for synthetic source bug.

Co-authored-by: Oleksandr Kolomiiets <[email protected]>
martijnvg added a commit to martijnvg/elasticsearch that referenced this pull request Sep 23, 2024
Added known issue entry for synthetic source bug.

Co-authored-by: Oleksandr Kolomiiets <[email protected]>
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.x
8.15

elasticsearchmachine pushed a commit that referenced this pull request Sep 23, 2024
Added known issue entry for synthetic source bug.

Co-authored-by: Oleksandr Kolomiiets <[email protected]>
elasticsearchmachine pushed a commit that referenced this pull request Sep 23, 2024
Added known issue entry for synthetic source bug.

Co-authored-by: Oleksandr Kolomiiets <[email protected]>
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.

3 participants