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.

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

add queue_durability property to amqp_consumer input #4628

Merged

Conversation

snubydev
Copy link
Contributor

@snubydev snubydev commented Sep 4, 2018

Required for all PRs:

  • [+] Signed CLA.
  • [+] Associated README.md updated.
  • [-] Has appropriate unit tests.

Added a Queue Durability property to amqp_consumer input plugin.

The amqp_consumer input plugin always creates at RabbitMQ server a queue with durable 'true',
that meas RabbitMQ is saving messages to disk in the queue util they are consumed. Sometimes it is needed to avoid impact of disk I/O and set the queue 'not durable'. It is easy to do in plugin configuration file.

Please review the code if it can be merged.

@danielnelson danielnelson added feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin area/rabbitmq labels Sep 4, 2018
@danielnelson danielnelson added this to the 1.8.0 milestone Sep 5, 2018
@danielnelson danielnelson merged commit a471497 into influxdata:master Sep 5, 2018
rgitzel pushed a commit to rgitzel/telegraf that referenced this pull request Oct 17, 2018
@snubydev snubydev deleted the feature/input-amqp_consumer branch March 2, 2019 13:33
otherpirate pushed a commit to otherpirate/telegraf that referenced this pull request Mar 15, 2019
otherpirate pushed a commit to otherpirate/telegraf that referenced this pull request Mar 15, 2019
dupondje pushed a commit to dupondje/telegraf that referenced this pull request Apr 22, 2019
athoune pushed a commit to bearstech/telegraf that referenced this pull request Apr 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/rabbitmq feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants