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

Queue.get() NPEs if queue doesn't exist #20

Open
chrismyang opened this issue Jun 17, 2013 · 2 comments
Open

Queue.get() NPEs if queue doesn't exist #20

chrismyang opened this issue Jun 17, 2013 · 2 comments

Comments

@chrismyang
Copy link

If a queue doesn't exist, Queue.get() throws an NPE – which is not a very helpful error. Actually, the semantics of queue existence is a little slippery, because from the API, it queues are treated sort of like namespaces for messages, rather than first-class things themselves (for example, you can't really create a queue other than posting a message to a queue).

At the very least, seems like you should replace the NPE with a more helpful message.

@edsrzf
Copy link
Contributor

edsrzf commented Jun 17, 2013

Thanks, I agree a different exception would be more helpful.

@chaplinkyle
Copy link

Today it doesn't throw a NPE, it throws an EmptyQueueException.

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

No branches or pull requests

3 participants