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

2014年 2月 27日 (木) 19:57:32 JST

Hello everybody,

I am Philo Bonianga, a systems engineer based in Osaka.
Please,  let me know if there are any tacit conditions for 
becoming a member of the Seasar development team.

Frankly speaking, I was interested in getting some role in the 
development team of Seasar but I was not sure I have enough 
time to fulfill such a responsability. 

This time, I am asking if I can get a role in the development team, 
like coding, debugging or assisting in some technical role such as 
investigating for potential bugs, reproducing and confirming 
reported bugs, etc…

Let me know if there are any conditions for participating in such 
work and what is the tacit contract with the Seasar Development 
body. Japanese language is OK.

Best regards,

===
Philo Bonianga <maeda at kodosystems.com>

On 2014/02/18, at 0:43, Hiroyuki Ohnaka <azusa at fieldnotes.jp> wrote:

> 大中です。
> 
> On Sat, 15 Feb 2014 21:33:22 +0900
> kubo <dbflute at gmail.com> wrote:
> 
>> 久保(jflute)です
>> 
>> MLの方の原因がそれかどうかまだちょっと理解していないのですが...
>> （二重に走って名前がAOPの名前がバッティング!?
>> 
>> 現状の実装は、たぶん違うクラスの初期化を
>> 同時に走らせたいるようにしてるんだと思われますが、
>> 一つにまとめるとそこがシリアルになっちゃうんじゃないかと。
> 
> On Mon, 17 Feb 2014 02:50:51 +0000
> YASUO HIGA <higa at isid.co.jp> wrote:
> 
>> ひがです。
>> 
>> 初期化が二重で走るのは、想定通りなんですが、初期化の中で、AOPの動的クラス作成が行われ、
>> そこがスレッドセーフになっていないのは、考慮漏れでした。
> 
> ああ、確かに全体でsynchronizedで囲むと複数のDaoの初回アクセスが
> 同時に走った時に待ちが発生しますね...
> 
> AOPのクラス拡張って、dao.diconのデフォルトだとクラスのエンハンスを
> 行わないので、(org.seasar.dao.impl.NullBeanEnhancer)、デフォルトだと
> 問題ない挙動だということになりますね、
> 
> (1) dao.dicon のBeanEnhancer関連の記述を元の設定に戻せないか
> (2) (1)が出来ない場合、DaoMetaDataFactoryImplでcreateDaoMetaDataを
>  オーバーライドしてもらう
> 
> 対応を提案したいと思います。
> 
> 振り回してすみませんが、よろしくお願いします。
> 
> 
> On Mon, 17 Feb 2014 02:50:51 +0000
> YASUO HIGA <higa at isid.co.jp> wrote:
> 
>> ひがです。
>> 
>> 初期化が二重で走るのは、想定通りなんですが、初期化の中で、AOPの動的クラス作成が行われ、
>> そこがスレッドセーフになっていないのは、考慮漏れでした。
>> 
>> createDaoMetaDataをsynchronizedするのが、良いんじゃないかと思います。
>> ________________________________________
>> 差出人: seasar-s2dao-dev-bounces at ml.seasar.org [seasar-s2dao-dev-bounces at ml.seasar.org] が次の人の代理で送信しました: Hiroyuki Ohnaka [azusa at fieldnotes.jp]
>> 送信日時: 2014年2月14日 21:15
>> 宛先: seasar-s2dao-dev at ml.seasar.org
>> 件名: [seasar-s2dao-dev:697] [Seasar-user:21792] について(DaoMetaDataの初期化タイミング)
>> 
>> 大中です。ご無沙汰しています。
>> 
>> [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のブロックを一つにまとめるように
>> 修正しようと思いますが、みなさんはどう思われますか？
>> 
>> 
>> 
>> _______________________________________________
>> seasar-s2dao-dev mailing list
>> seasar-s2dao-dev at ml.seasar.org
>> https://ml.seasar.org/mailman/listinfo/seasar-s2dao-dev
>> _______________________________________________
>> seasar-s2dao-dev mailing list
>> seasar-s2dao-dev at ml.seasar.org
>> https://ml.seasar.org/mailman/listinfo/seasar-s2dao-dev
> 
> 
> -- 
> azusa at gmail <azusa at fieldnotes.jp>
> _______________________________________________
> seasar-s2dao-dev mailing list
> seasar-s2dao-dev at ml.seasar.org
> https://ml.seasar.org/mailman/listinfo/seasar-s2dao-dev