Ist die von TheadFactoryBuilder generierte ThreadFactory sicher?

Question

Angenommen, ich habe den folgenden Code und makePool kann von mehreren Threads aufgerufen werden. Ich verwende ThreadFactoryBuilder, um sicherzustellen, dass die Threads einen eindeutigen Namen haben.

private final ThreadFactory threadFactory = new ThreadFactoryBuilder().setNameFormat(
        "MyBusinessClass-%d").build(); 

public ExecutorService makePool() { 
    return Executors.newFixedThreadPool(1, threadFactory); 
} 

oder muss ich den Pool anzunehmen, ist nicht Thread-sicher, und haben so etwas wie zu tun:

private static final AtomicLong poolCount = new AtomicLong(0L); 

public ExecutorService makePool() { 
    private final ThreadFactory threadFactory = new ThreadFactoryBuilder().setNameFormat(
     "MyBusinessClass- " + poolCount.incrementAndGet() + "-%d").build(); 
    return Executors.newFixedThreadPool(1, threadFactory); 
} 

Answer 1

Meine Lektüre des source code (Guava 10.0.1) ist, dass es Thread sicher, solange die Rückfadenfabrik fadensicher ist. In Ihrem Code ist die Backing-Thread-Factory eine Instanz von Executors.DefaultThreadFactory, die Thread-sicher ist.

Beachten Sie, dass keines dieser angegeben ist ... soweit ich .. aber sehen kann, ist die Art und Weise, dass diese Klassen codiert sind, sagt mir, dass sie thread-safe nach Design sind. Die veränderbaren Komponenten der jeweiligen Objekte bestehen aus Zählern, die als Atomics implementiert sind, und die Thread-Erzeugung/Start hat eine passiert-vor.