Skip to content

Remove tail cleanup call to avoid double decrement#6089

Merged
danielnelson merged 1 commit intomasterfrom
remove-tail-cleanup
Jul 8, 2019
Merged

Remove tail cleanup call to avoid double decrement#6089
danielnelson merged 1 commit intomasterfrom
remove-tail-cleanup

Conversation

@danielnelson
Copy link
Contributor

@danielnelson danielnelson commented Jul 8, 2019

This is discussed further on influxdata/tail#6. In short, over time the Cleanup call has been broken and shouldn't be used since it decrements the reference count unconditionally.

closes: #3573

hpcloud/tail#153

Required for all PRs:

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

@danielnelson danielnelson added the fix pr to fix corresponding bug label Jul 8, 2019
@danielnelson danielnelson added this to the 1.11.2 milestone Jul 8, 2019
@danielnelson danielnelson merged commit 601f499 into master Jul 8, 2019
@danielnelson danielnelson deleted the remove-tail-cleanup branch July 8, 2019 21:48
danielnelson added a commit that referenced this pull request Jul 9, 2019
bitcharmer pushed a commit to bitcharmer/telegraf that referenced this pull request Oct 18, 2019
athoune pushed a commit to bearstech/telegraf that referenced this pull request Apr 17, 2020
idohalevi pushed a commit to idohalevi/telegraf that referenced this pull request Sep 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix pr to fix corresponding bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Logparser plugin don't process new lines after telegraf configuration reload.

2 participants