Skip to content

chore(parsers.influx): Cleanup code#18951

Open
srebhan wants to merge 1 commit into
influxdata:masterfrom
srebhan:parser_influx
Open

chore(parsers.influx): Cleanup code#18951
srebhan wants to merge 1 commit into
influxdata:masterfrom
srebhan:parser_influx

Conversation

@srebhan
Copy link
Copy Markdown
Member

@srebhan srebhan commented May 20, 2026

Summary

Cleanup code.

Checklist

Related issues

@telegraf-tiger telegraf-tiger Bot added chore plugin/parser 1. Request for new parser plugins 2. Issues/PRs that are related to parser plugins labels May 20, 2026
@telegraf-tiger
Copy link
Copy Markdown
Contributor

@srebhan srebhan 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 May 20, 2026
Copy link
Copy Markdown
Contributor

@skartikey skartikey left a comment

Choose a reason for hiding this comment

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

@srebhan Thanks! one minor comment.

@@ -195,7 +193,7 @@ func NewStreamParser(r io.Reader) *StreamParser {
// SetTimeFunc changes the function used to determine the time of metrics
// without a timestamp. The default TimeFunc is time.Now. Useful mostly for
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.

Tiny nit: with TimeFunc gone, the comment on StreamParser.SetTimeFunc still says "The default TimeFunc is time.Now". Worth rewording to drop the dangling type reference.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore plugin/parser 1. Request for new parser plugins 2. Issues/PRs that are related to parser plugins 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.

2 participants