[seasar-s2dao-dev:697] [Seasar-user:21792] について(DaoMetaDataの初期化タイミング)

[Hiroyuki Ohnaka]

大中です。ご無沙汰しています。

 [Seasar-user:21792] で小林さんに回答してもらったDaoMetaDataFactoryImplの初期化処理ですが、

DaoMetaDataFactoryImpl#getDaoMetaDataでいったんdaoMetaDataCacheで
排他をかけていますが、その後の128行目から130行目のところは、
排他がいったん外れるため、初期化処理が二重に走る場合があるように
思われます。

https://github.com/seasarorg/s2dao/blob/master/s2-dao/src/main/java/org/seasar/dao/impl/DaoMetaDataFactoryImpl.java#L128-L130

この部分は、2つあるsynchronizedのブロックを一つにまとめるように
修正しようと思いますが、みなさんはどう思われますか？