Skip to content

feat(processors.parser): Add strategy to merge parsed metric with parent individually#18558

Merged
srebhan merged 4 commits into
influxdata:masterfrom
srebhan:processors_parser_issue_16382a
Apr 17, 2026
Merged

feat(processors.parser): Add strategy to merge parsed metric with parent individually#18558
srebhan merged 4 commits into
influxdata:masterfrom
srebhan:processors_parser_issue_16382a

Conversation

@srebhan

@srebhan srebhan commented Mar 17, 2026

Copy link
Copy Markdown
Member

Summary

This PR adds two new merge strategies to merge each parsed metric individually with its parent to preserve previously existing fields and tags.

Checklist

Related issues

resolves #16382

@telegraf-tiger telegraf-tiger Bot added fix pr to fix corresponding bug plugin/processor labels Mar 17, 2026
@srebhan srebhan changed the title fix(processors.parser): Add strategy to merge parsed metric with parent individually feat(processors.parser): Add strategy to merge parsed metric with parent individually Mar 19, 2026
@telegraf-tiger telegraf-tiger Bot added the feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin label Mar 19, 2026
@srebhan srebhan removed the fix pr to fix corresponding bug label Mar 19, 2026

@mstrandboge mstrandboge left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code looks great, just a couple comments on the documentation

Comment thread plugins/processors/parser/README.md
Comment thread plugins/processors/parser/README.md

@skartikey skartikey left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@srebhan Thanks!

@skartikey skartikey removed their assignment Apr 13, 2026
@skartikey skartikey added the ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review. label Apr 13, 2026
@srebhan srebhan assigned srebhan and unassigned mstrandboge Apr 17, 2026
@srebhan srebhan force-pushed the processors_parser_issue_16382a branch from e68967e to 7b25354 Compare April 17, 2026 09:08
@telegraf-tiger

Copy link
Copy Markdown
Contributor

@srebhan srebhan merged commit b20a62d into influxdata:master Apr 17, 2026
27 checks passed
@srebhan srebhan deleted the processors_parser_issue_16382a branch April 17, 2026 10:00
@github-actions github-actions Bot added this to the v1.39.0 milestone Apr 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin plugin/processor ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[[processors.parser]] merge with more than one parsed metric does not work

3 participants