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.

Sign up for GitHub

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

Jump to bottom

Fix parsing of multiple files with different headers (#6318). #8400

Merged
merged 1 commit into from
Nov 13, 2020
Merged

Fix parsing of multiple files with different headers (#6318). #8400

merged 1 commit into from
Nov 13, 2020

Conversation

srebhan
Copy link
Member

@srebhan srebhan commented Nov 13, 2020

Required for all PRs:

  • Signed CLA.
  • Associated README.md updated.
  • Has appropriate unit tests.

Alternative to PR #6324 also fixing all other plugins (e.g. tail). This resolves issue #6318 in my setup.

@srebhan srebhan added the fix pr to fix corresponding bug label Nov 13, 2020
@srebhan srebhan requested a review from ssoroka November 13, 2020 10:10
ssoroka
ssoroka approved these changes Nov 13, 2020
View reviewed changes
Copy link
Contributor

@ssoroka ssoroka left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Looks good

@ssoroka ssoroka merged commit 049daf7 into influxdata:master Nov 13, 2020
ssoroka pushed a commit that referenced this pull request Nov 13, 2020
@srebhan @ssoroka
Fix parsing of multiple files with different headers (#6318). (#8400)
7b71a68 
(cherry picked from commit 049daf7)
@srebhan srebhan deleted the fix-6318 branch November 13, 2020 16:59
@ssoroka ssoroka mentioned this pull request Nov 13, 2020
Closed
@srebhan srebhan mentioned this pull request Jun 7, 2022
Closed
@srebhan srebhan mentioned this pull request Oct 26, 2022
Closed
arstercz pushed a commit to arstercz/telegraf that referenced this pull request Mar 5, 2023
@srebhan
Fix parsing of multiple files with different headers (influxdata#6318)…
1c81249 
…. (influxdata#8400)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ssoroka ssoroka ssoroka approved these changes

Assignees
No one assigned
Labels
fix pr to fix corresponding bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

input.file plugin behavior with different input metrics files
2 participants
@srebhan @ssoroka