Keisuke.K

[CORE-216] 複数行 SQL を実行できるよう修正。…
[CORE-216] 複数行 SQL を実行できるよう修正。

これにより、行コメントが正しく処理できるようになる。

+1

+1

[CORE-96] enqueue 待ちを最大1分に拡大。
[CORE-96] enqueue 待ちを最大1分に拡大。
  • More
  • CR-22
  • summarized and closed
to: yamkazu この内容でどうでしょ?

to: yamkazu
この内容でどうでしょ?

特に大きな意図はないです。

特に大きな意図はないです。

だいちゃんも言ってますが、DefaultCheckConstraintModel を作るのであれば DefaultCheckConstraintModelBuilder を作る時点で CheckConstraintModelBuilder 型で受け取る必要性が感じられないです。 あと実装の問題もあって、どの時点から Abstract Builder の VO 型を決定付けようか、という点。 ...

だいちゃんも言ってますが、DefaultCheckConstraintModel を作るのであれば DefaultCheckConstraintModelBuilder を作る時点で CheckConstraintModelBuilder 型で受け取る必要性が感じられないです。
あと実装の問題もあって、どの時点から Abstract Builder の VO 型を決定付けようか、という点。
現在の実装だと、例えば DefaultForeignKeyConstraintModelBuilder の階層は
ValueObjectBuilder -> ConstraintModelBuilder -> KeyConstraintModelBuilder -> ForeignKeyConstraintModelBuilder -> DefaultForeignKeyConstraintModelBuilder
になっていますが、DefaultPrimaryKeyConstraintModel の階層は
ValueObjectBuilder -> ConstraintModelBuilder -> KeyConstraintModelBuilder -> DefaultPrimaryKeyConstraintModelBuilder
となっています。
ここで、KeyConstraintModelBuilder の時点で VO 型を決定付けると、ForeignKeyConstraintModelBuilder#setXxx() の addConfigurator(new BuilderConfigurator<S>(){}) になっている部分の S が変えられようにも変えられないんですよ。
DefaultPrimaryKeyConstraintModelBuilder の前に PrimaryKeyConstraintModelBuilder を作れと言ってしまえばそれまでなんですが、実装が何もないクラスになります。
中身が空っぽのクラスがざっと増えるのはちょっと、と思ったので作らなかったんですが、どう思います?

  • More
  • CR-20
  • finished reviewing
[COM-1] 受動態撲滅。
[COM-1] 受動態撲滅。
[COM-1] Javadoc修正。
[COM-1] Javadoc修正。
[COM-1] Javadoc修正。
[COM-1] Javadoc修正。
[COM-1] Javadoc修正。
[COM-1] Javadoc修正。
[COM-1] Javadoc整備。
[COM-1] Javadoc整備。
[COM-1] 更新系のテストを追加。他、Javadoc の修正および不具合修正。
[COM-1] 更新系のテストを追加。他、Javadoc の修正および不具合修正。
[COM-1] 更新時の処理も対応。他、パッチ適用およびレビュー時のコメントを参考に修正。
[COM-1] 更新時の処理も対応。他、パッチ適用およびレビュー時のコメントを参考に修正。
update や insert の返却値は更新行数 (int) です。 ここから別インタフェースを定義する、というのがいまいちよくわかりません。

update や insert の返却値は更新行数 (int) です。
ここから別インタフェースを定義する、というのがいまいちよくわかりません。

例えば UPDATE とかです。現状 SELECT だけしか意識していないので handleResultSet しかありません。別インタフェースってのはファイルを分けるという意味でいいでしょうか。それならば、trunk 上では既に分けています。

例えば UPDATE とかです。現状 SELECT だけしか意識していないので handleResultSet しかありません。別インタフェースってのはファイルを分けるという意味でいいでしょうか。それならば、trunk 上では既に分けています。

XOR です。

XOR です。

将来的に ResultSet 以外のものも handle するかもしれないので、SqlExecutor の処理を handle するという意味で、この名前にしてます。

将来的に ResultSet 以外のものも handle するかもしれないので、SqlExecutor の処理を handle するという意味で、この名前にしてます。

[COM-1] ハンドラのメソッド名を変更。
[COM-1] ハンドラのメソッド名を変更。
[COM-1] コンパイルエラー修正
[COM-1] コンパイルエラー修正
[COM-1] execute 時に SQL 文やハンドラを指定するよう変更。…
[COM-1] execute 時に SQL 文やハンドラを指定するよう変更。

また、致命的なバグを修正。

[CORE-179] パッチの適用。
[CORE-179] パッチの適用。
[COM-1] 各種修正。…
[COM-1] 各種修正。

・ハンドラの setter を削除。

・コンストラクタの種類を増し増し。

・オートコミットの記述箇所を修正。

[COM-1] レビュー内容を反映。
[COM-1] レビュー内容を反映。
必要そうですね。直します。

必要そうですね。直します。

  • More
  • CR-24
  • finished reviewing
[CORE-179] 不足分のビルダークラスを作成。
[CORE-179] 不足分のビルダークラスを作成。
[CORE-179] apply に渡す ValueObject は、ビルド対象の ValueObject のみ受け入れるよう戻す。
[CORE-179] apply に渡す ValueObject は、ビルド対象の ValueObject のみ受け入れるよう戻す。
[COM-26] ReferenceListener を一般的なリスナのコードに変更。
[COM-26] ReferenceListener を一般的なリスナのコードに変更。