Skip to content

Conversation

@jsoriano
Copy link
Member

@jsoriano jsoriano commented Oct 9, 2023

Handle elastic/integrations#8115 (comment), where nested objects are stored inside an array of arrays.

@jsoriano jsoriano self-assigned this Oct 9, 2023
@jsoriano jsoriano requested a review from a team October 9, 2023 14:43
@jsoriano jsoriano marked this pull request as draft October 9, 2023 14:45
@jsoriano
Copy link
Member Author

jsoriano commented Oct 9, 2023

Going back to draft by now in case this is not needed.

@jsoriano jsoriano force-pushed the arrays-of-arrays-of-nested branch from cc5ddff to edb4069 Compare October 9, 2023 14:51
@jsoriano
Copy link
Member Author

jsoriano commented Oct 9, 2023

Closing this as the only case we had with this has been removed in elastic/integrations#8115, thanks @marc-gr!

@jsoriano jsoriano closed this Oct 9, 2023
@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

cc @jsoriano

@mrodm mrodm reopened this Oct 10, 2023
Copy link
Contributor

@mrodm mrodm left a comment

Choose a reason for hiding this comment

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

LGTM

@mrodm mrodm marked this pull request as ready for review October 10, 2023 14:02
@mrodm mrodm merged commit f1b90c7 into elastic:main Oct 10, 2023
jsoriano added a commit that referenced this pull request Sep 9, 2024
We added some additional validations for subobjects and arrays of objects
in #1498, #1489 and related PRs. These validations only apply to packages
with spec starting on 3.0.1.

These validations rely on the structure of documents. With the adoption of
features like `subobjects: false` or synthetic source the structure is lost, and
exceptions based on spec version are not working, so the tests fail for cases
where they should not for versions of the spec older than 3.0.1. This happens
for example in the `dns` data stream of the `network_traffic` package when
LogsDB is enabled.
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.

3 participants