Параллельный доступ к unmodifiableMap

Question

@Singleton 
@LocalBean 
@Startup 
@ConcurrencyManagement(ConcurrencyManagementType.BEAN) 
public class DeliverersHolderSingleton { 

    private volatile Map<String, Deliverer> deliverers; 

    @PostConstruct 
    private void init() { 
     Map<String, Deliverer> deliverersMod = new HashMap<>(); 
     for (String delivererName : delivererNames) { 
      /*gettig deliverer by name*/ 
      deliverersMod.put(delivererName, deliverer); 
     } 
     deliverers = Collections.unmodifiableMap(deliverersMod); 
    } 

    public Deliverer getDeliverer(String delivererName) { 
     return deliverers.get(delivererName); 
    } 

    @Schedule(minute="*", hour="*") 
    public void maintenance() { 
     init(); 
    } 
} 

Singleton используется для хранения данных. Данные обновляются один раз в минуту. Возможно ли, что чтение из unmodifiableMap будет проблемой с синхронизацией? Возможно ли, что произойдет переупорядочение в методе init и будет опубликована ссылка на сборник, но сбор не заполняется полностью?

Answer 1

Модель памяти Java гарантирует, что there is a happens-before relationship between a write and a subsequent read to a volatile variable. Другими словами, если вы пишете переменную volatile и впоследствии читаете эту же переменную, у вас есть гарантия, что операция записи будет видна, даже если задействовано несколько потоков:

Запись в нестабильное поле (§8.3.1.4) - до каждого последующего чтения этого поля.

Он идет дальше и гарантирует, что любая операция, которая была до операции записи также будет видна в момент чтения (благодаря правилу порядка программы, и тот факт, что происходит, прежде, чем отношение транзитивности).

Ваш метод getDeliverers читается из изменчивой переменной, поэтому он увидит последнюю запись, записанную на линии deliverers = Collections.unmodifiableMap(deliverersMod);, а также предыдущие операции, в которых заполняется карта.

Таким образом, ваш код является потокобезопасным, и ваш метод getDeliverers вернет результат на основе последней версии вашей карты.

Answer 2

вопросы безопасности Thread здесь:

• несколько операций чтения из HashMap - это поточно, потому что несколько операций чтения допускаются до тех пор, пока нет изменений в коллекции и записывает в HashMap не произойдет, потому что карта является unmodifiableMap()

• чтения/записи на deliverers - это поточно, потому что все ссылки на Java назначения являются неделимыми

Здесь я не вижу никаких небезопасных операций.

Я хотел бы отметить, что имя метода init() вводит в заблуждение, это предполагает, что он вызывается один раз во время инициализации; Я бы предложил позвонить ему rebuild() или recreate().

Answer 3

Согласно Переупорядочению сетки здесь http://g.oswego.edu/dl/jmm/cookbook.html, то первая операция является Normal Store не может быть заказан с второй операцией будучи Volatile Store, так что в вашем случае, до тех пор, как неизменная карта не равно нуль, то не было бы каким-либо переназначение проблемы.

Кроме того, все записи, которые происходят до энергозависимого хранилища, будут видны, поэтому вы не увидите никаких проблем с публикацией.