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

[pkg/ottl] fix handling of nested maps within slices in flatten function #36204

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

bacherfl
Copy link
Contributor

@bacherfl bacherfl commented Nov 5, 2024

Description

This PR fixes the behavior of the flatten function to also modify nested objects within slices. Currently WIP, as #36198 should be merged first to avoid too many conflicts

Link to tracking issue

Fixes #36162

Testing

Added unit test and adapted e2e tests

# Conflicts:
#	pkg/ottl/ottlfuncs/func_flatten.go
Signed-off-by: Florian Bacher <[email protected]>
@bacherfl bacherfl marked this pull request as ready for review November 19, 2024 11:55
@bacherfl bacherfl requested a review from a team as a code owner November 19, 2024 11:55
Copy link
Contributor

@evan-bradley evan-bradley left a comment

Choose a reason for hiding this comment

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

Looks good to me, sorry for the long delay with a review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[pkg/ottl] unexpected behavior of flatten function for slices containing nested objects
3 participants