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

513,304
74
731
1,106

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

7,785
3
31
45

Java: Synchronize an ExecutorService necessary?

2 Answers2