why did you change it to add? Just to fool checkstyle/findbugs because you are not doing anything if the joinrequest could not be placed because the q is full? Doesn't it make more sense to do something sensible, e.g. log a warning. It seems to me that it quite a severe problem if the queue is full. Could also be that I'm wrong.