I have a multi-threaded application where a thread sends a message to another thread. The waiting thread polls for the message and reacts (locks are handled). Like this:
You're doing busy waiting. You should never do that because it wastes CPU cycles. A thread or process waiting for some event should be in blocked state. Here are possible ways to achieve this:
Have you tried to put some Thread.sleep to avoid executing the while loop constantly? It will liberate CPU for your other threads and avoid hogs
http://docs.oracle.com/javase/tutorial/essential/concurrency/sleep.html
The correct way to avoid this is to use the wait/notify mechanism implemented by java.lang.Object
, or one of the higher level concurrency mechanisms provided by the Java class libraries:
(Pick the mechanism that is the best match for what your specific use-case ...)
Using Thread.sleep
is not a good solution. While you reduce CPU load (compared with a polling loop), the flip-side is that you reduce responsiveness.
I'm using a BlockingQueue now. But maybe I'm doing it incorrectly. I just added the actually code above. Do you see my problem?
Yea. You are using the queue in a way that is designed to avoid blocking. That's the wrong approach. You should use take()
(which will block until an entry becomes available) instead of poll()
, and get rid of the code that tests the queue size.
Your "killMeMailbox" stuff seems to be designed to allow you to stop waiting for mail. You should be able to implement that using Thread.interrupt
. (An interrupt will unblock a take()
call ...)
If you can reformulate your problem in terms of task-executor then consider using a SingleThreadExecutor. If you need something more exotic - either a concurrent queue or even wait()/notify().