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

Inspect message and consumer counts for a queue. #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Inspect message and consumer counts for a queue. #16

wants to merge 1 commit into from

Conversation

phensley
Copy link

@phensley phensley commented Jun 24, 2016

re #34

@kron4eg
Copy link
Contributor

kron4eg commented Jun 24, 2016

@phensley Thank you!
Looks good to me but before merge I'd like to see in what context this will be used. Could you please provide more background on this?

@phensley
Copy link
Author

The use case we have is for a daemon to be aware of a queue's current size, and whether there are > 0 consumers, and be able to take action.

@kron4eg
Copy link
Contributor

kron4eg commented Jun 25, 2016

@phensley would it make it for you if cony will expose current *amqp.Connection through some getter?

@phensley
Copy link
Author

@kron4eg You could export Connection() (*amqp.Connection, error) and I could move the QueueInspect method into my own code.

@kron4eg
Copy link
Contributor

kron4eg commented Jun 25, 2016

@phensley yeah, that's what I meant.

@phensley
Copy link
Author

@kron4eg Works for me.

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

Successfully merging this pull request may close these issues.

3 participants