-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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] Add merge_all_maps #32954
Comments
Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
@TylerHelmuth How would you feel about changing the |
We could add a third, optional parameter like Technically the solution to this issue right now is to have multiple |
I think the issue is that with things like Windows event data there are an arbitrary number of maps present in the array |
@strawgate interesting, can you go into more details about that use case? |
I detailed it here: #32952 Happy to include more info if you have specific questions |
This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
This use case still exists |
I would be willing to make an attempt at contributing this if it seems reasonable? |
This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
Component(s)
No response
Is your feature request related to a problem? Please describe.
Right now merge_maps requires a source map and a target map.
There does not seem to be a good way to merge an array of maps.
Describe the solution you'd like
merge_all_maps(source, target, method)
where source is an array of maps and target is a map
Method (or whatever it's called on merge_map) would decide whether it's an insert, upset, etc
Describe alternatives you've considered
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: