Java: Synchronize an ExecutorService necessary?

Question

I have a class containing an ExecutorService that can be shared between threads:

class MyExecutor {
    ExecutorService e = Executors.newSingleThreadExecutor();
    ....
    .... 
    public void add(Runnable r) {
         e.executre(r);
    } 
}

Is it necessary to synchronize the ExecutorService object in the add method since the add method can be called from differens threads or is the ExecutorService thread safe?

Possible duplicate of http://stackoverflow.com/questions/1702386/is-threadpoolexecutor-thread-safe? — THelper, Dec 09 '11 at 13:58

score 5 · Accepted Answer · answered Dec 09 '11 at 14:06

5

ExecutorService has to use a thread safe queue (Which it does by default). This is all that is needed.

answered Dec 09 '11 at 14:06

Peter Lawrey

525,659
79
751
1,130

score 2 · Answer 2 · answered Dec 09 '11 at 13:59

2

No, there is no need to synchronize calls to add() method.

answered Dec 09 '11 at 13:59

maximdim

8,041
3
33
48

Java: Synchronize an ExecutorService necessary?

2 Answers2