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

[YASUO HIGA]

Hi Philo,

Thanks for your offer.
However, we don't do new development for several years, so there are few tasks about the development.
I would appreciate it if you could tell the goodness of Seasar to the people around you.

Thanks,

Yasuo
________________________________________
差出人: seasar-s2dao-dev-bounces ＠ ml.seasar.org [seasar-s2dao-dev-bounces ＠ ml.seasar.org] が次の人の代理で送信しました: philo ＠ galaxy.ocn.ne.jp [philo ＠ galaxy.ocn.ne.jp]
送信日時: 2014年2月27日 19:57
宛先: seasar-s2dao-dev ＠ ml.seasar.org
件名: [seasar-s2dao-dev:701] Re: [Seasar-user:21792] について(DaoMetaDataの初期化タイミング)

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 ＠ kodosystems.com>


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

> 大中です。
>
> On Sat, 15 Feb 2014 21:33:22 +0900
> kubo <dbflute ＠ gmail.com> wrote:
>
>> 久保(jflute)です
>>
>> MLの方の原因がそれかどうかまだちょっと理解していないのですが...
>> （二重に走って名前がAOPの名前がバッティング!?
>>
>> 現状の実装は、たぶん違うクラスの初期化を
>> 同時に走らせたいるようにしてるんだと思われますが、
>> 一つにまとめるとそこがシリアルになっちゃうんじゃないかと。
>
> On Mon, 17 Feb 2014 02:50:51 +0000
> YASUO HIGA <higa ＠ 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 ＠ isid.co.jp> wrote:
>
>> ひがです。
>>
>> 初期化が二重で走るのは、想定通りなんですが、初期化の中で、AOPの動的クラス作成が行われ、
>> そこがスレッドセーフになっていないのは、考慮漏れでした。
>>
>> createDaoMetaDataをsynchronizedするのが、良いんじゃないかと思います。
>> ________________________________________
>> 差出人: seasar-s2dao-dev-bounces ＠ ml.seasar.org [seasar-s2dao-dev-bounces ＠ ml.seasar.org] が次の人の代理で送信しました: Hiroyuki Ohnaka [azusa ＠ fieldnotes.jp]
>> 送信日時: 2014年2月14日 21:15
>> 宛先: seasar-s2dao-dev ＠ 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 ＠ ml.seasar.org
>> https://ml.seasar.org/mailman/listinfo/seasar-s2dao-dev
>> _______________________________________________
>> seasar-s2dao-dev mailing list
>> seasar-s2dao-dev ＠ ml.seasar.org
>> https://ml.seasar.org/mailman/listinfo/seasar-s2dao-dev
>
>
> --
> azusa ＠ gmail <azusa ＠ fieldnotes.jp>
> _______________________________________________
> seasar-s2dao-dev mailing list
> seasar-s2dao-dev ＠ ml.seasar.org
> https://ml.seasar.org/mailman/listinfo/seasar-s2dao-dev

_______________________________________________
seasar-s2dao-dev mailing list
seasar-s2dao-dev ＠ ml.seasar.org
https://ml.seasar.org/mailman/listinfo/seasar-s2dao-dev