[Seasar-user:21682] Re: S2JDBC の AutoSelect への includes() と excludes() 実装

[Toshihiro Nakamura]

中村(taedium)です。

修正おつかれさまです。

s2jdbc-itについて、1つ誤ってコミットされたと思われる箇所があります。
env_ut.txtの内容が「${database}」に変更されてしまっていますが、
元の値は、svn checkoutした状態でJUnitから動かせるように「hsqldb」でした。

# Jenkins上では起動時のパラメータで上書きしているので
# ITとしての動作に問題はでていません。


2013年7月22日 17:45 Koichi Kobayashi <koichik ＠ improvement.jp>:

> 小林 (koichik) です．
>
> 確認ありがとうございました。
> # Jenkinsでのs2jdbc-itがOracleやDB2でも
> # うまく動いてますね。
>
> 念のため軽く見てもらえますか？＞ひがさん、中村さん
>
>
> On Sun, 21 Jul 2013 23:14:44 +0900, Jun Futagawa <jfut ＠ integ.jp> wrote:
>
> > ふたがわ (jfut) です。
> >
> > On 2013/07/20 11:06, Koichi Kobayashi wrote:
> >
> > > パッチを取り込んでコミットしました。
> > > スナップショットをデプロイしたのでご確認ください。
> >
> > 取り込みありがとうございます。
> > スナップショットで動作確認し、問題ありませんでした。
> >
> > > パッチを適用してから気づいたのですが、重複する
> > > 機能として既に eager() がありました。
> > > # すっかり忘れてました。。。 心より恥じる。
> > ...
> > > これは落とし穴にしか見えないので、includes() を
> > > 指定した場合は eager() を指定しなくても済むように
> > > 修正して、ドキュメントには両方を混ぜて使わないよう
> > > 記載しました。
> > > # eager() 本体の説明より注意点が長くなった。。。
> > > # でも混ぜて使わなければややこしいことはないはず
> >
> > 使い方が違うので気にしていませんでした(すいません)が、
> > 説明していただいた上記の挙動が判り易いと思いますので
> > 入れていただいた修正でより良いと思います。
> >
> > 一点、大した違いはないですが、@Id が付いたプロパティは
> > 無条件で対象なので、下記のように、final boolean lazy の値を
> > 取得する前で return true; しても良いのかなと思いました。
> >
> >         if (propertyMeta.isId()) {
> >             return true;
> >         }
> >         final boolean lazy = isLazy(propertyMeta, joinMeta);
> >
> > > ドキュメントの includes/excludes 追加位置も eager() の
> > > 直前にしました。
> > >
> > > 些細なことですが、ドキュメントなどで includes() 等が
> > > join() より前にあったのですが、includes() の指定は
> > > join() で指定した関連名を使えるので、後の方に移動
> > > しました。
> > > # SQL 書く時も SELECT 句は後で書く方が楽だし。
> >
> > これはどちらを先に指定するか迷って、SQL を書くときは
> > カラム指定が先で join 指定が後なので、その順番の例で
> > 書いていました。どちらに書いてあっても問題はないですし、
> > 先頭から流れるように書いていくときは先に join があった方が
> > 判り易いので、書いていただいたままで良いと思います。
> >
> > ドキュメントの補足、ありがとうございます。
> >
> > --
> > Jun Futagawa
> > _______________________________________________
> > Seasar-user mailing list
> > Seasar-user ＠ ml.seasar.org
> > https://ml.seasar.org/mailman/listinfo/seasar-user
>
>
> --
> {
>   name: "Koichi Kobayashi",
>   mail: "koichik ＠ improvement.jp",
>   blog: "http://d.hatena.ne.jp/koichik/",
>   twitter: "@koichik"
> }
>
> _______________________________________________
> Seasar-user mailing list
> Seasar-user ＠ ml.seasar.org
> https://ml.seasar.org/mailman/listinfo/seasar-user
>



-- 
Toshihiro Nakamura
-------------- next part --------------
HTMLの添付ファイルを保管しました...
URL: <http://ml.seasar.org/archives/seasar-user/attachments/20130722/c1f88eb7/attachment.html>