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

AMQP as input nevers ack it's messages.. #7

Closed
ThomasADavis opened this issue Sep 5, 2017 · 5 comments
Closed

AMQP as input nevers ack it's messages.. #7

ThomasADavis opened this issue Sep 5, 2017 · 5 comments
Labels

Comments

@ThomasADavis
Copy link

In testing, I found that if you use AMQP (RabbitMQ) as an input, the messages will never get ACK'd in rabbitmq, which causes a backlog to build.

In the AMQP input file, you've turned off AutoACK, but never ack any messages as being delivered.

You also never set a QOS, which then defaults to 0 in the rabbitmq. This also could cause problems if multiple benthos are sharing the same queue, or performance problems in heavy queue traffic.

@Jeffail Jeffail added the bug label Sep 15, 2017
@Jeffail
Copy link
Collaborator

Jeffail commented Sep 15, 2017

Hey @ThomasADavis, sorry for the delayed response, this issue slipped under my radar. I'll take a look next time I get a chance, or I'd be happy to take a pull request.

I'm still fairly unfamiliar with AMQP so thanks for highlighting these issues.

@Jeffail
Copy link
Collaborator

Jeffail commented Sep 17, 2017

Hey @ThomasADavis, had a quick moment this afternoon to push a fix for this but haven't got time to test today. If you give it a try then let me know how it goes.

@sukria
Copy link

sukria commented Sep 19, 2017

Hey there. I'm also hit by this bug, I'm going to give your fix a test.

@ThomasADavis
Copy link
Author

I will not have time for several weeks to work on this. A quick glance at the code it does appears to be correct.

@sukria
Copy link

sukria commented Sep 19, 2017

Works for me! Thanks for the fix.
Just tested it, and my queue got consumed entirely.

@Jeffail Jeffail closed this as completed Sep 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants