[Seasar-user:21238] Re: Domaでquery.isBatchSupported()がfalseのときbatch insertで期待する値が返らない

[Toshihiro Nakamura]

中村です。

報告ありがとうございます。
ご指摘のとおりDomaの不具合でした。
修正してSNAPSHOT作成しましたので
よければお試しください。

http://maven.seasar.org/maven2-snapshot/org/seasar/doma/doma/1.20.1-SNAPSHOT/doma-1.20.1-20111224.125038-2.jar

2011年12月24日16:51 Taichi Uragami <[E-MAIL ADDRESS DELETED]>:
> うらがみ（backpaper0）と申します。
> 表題の通り、Domaのbatch insertで期待する値が返らない場合がありました。
> 再現したコードはこちらになります。https://github.com/backpaper0/ormappers/blob/master/src/test/java/sample/DomaSample.java#L77
> 上記再現コードで使用しているUserエンティティは次のようなコードになっています。https://github.com/backpaper0/ormappers/blob/master/src/main/java/sample/doma/entity/User.java
> またUserDaoは次のようになっています。https://github.com/backpaper0/ormappers/blob/master/src/main/java/sample/doma/dao/UserDao.java
> 最初に示した再現コードの84行目、85行目でどちらも1となることを期待するのですが実際の値は0となっていました。
> Domaのソースを見ましたが、BatchInsertCommandクラスのexecuteInternalメソッドでquery.isBatchSupported()がfalseを返すときの処理で戻り値として使用されているローカル変数updatedRowsに値が設定されていないことが原因だと思います。
> http://www.seasar.org/source/browse/doma/trunk/doma/src/main/java/org/seasar/doma/internal/jdbc/command/BatchInsertCommand.java?revision=616&view=markup#l38
> forループ内でコールされているexecuteUpdateの戻り値をupdatedRowsの各要素に代入するのが正しい動作だと思うのですが、そうであれば修正をお願い致します。
> --
> うらがみ
> blog: http://d.hatena.ne.jp/backpaper0/
> twitter: @backpaper0
> _______________________________________________
> Seasar-user mailing list
> [E-MAIL ADDRESS DELETED]
> https://ml.seasar.org/mailman/listinfo/seasar-user



-- 
Toshihiro Nakamura