These are chat archives for scalikejdbc/ja

27th
Mar 2016
Tsuyoshi Yoshizawa
@tsuyoshizawa
Mar 27 2016 13:42
HikariCP のコネクションプーリングサポートが ScalikeJDBC になかったので追加してみようかと思ったんですが、 ConnectionPoolSettings に含まれている以上のものを僕自身が設定することが多く困ってしまいました。 ScalikeJDBC のほうでコネクションプーリングの設定に拡張性を持たせようとする計画があったり、してもよいと思ったりしますか?
イメージ的には Play のような設定ができれば柔軟に各コネクションプーリングライブラリの設定ができるようになるのかなと思っています。
db {
  default {
    driver =
    url =
    username =
    password =
    hikaricp {
      // HikcariCP 専用のプロパティ
    }
    dbcp2 {
      // Commons DBCP2 専用のプロパティ
    }
  }
}
Kazuhiro Sera
@seratch
Mar 27 2016 13:56
まず db.default.hikaricp みたいなネームスペースは今の所考えてません。
HikariCP は DataSourceClassName をはじめとして独自属性が結構ありますけど、DataSource をつくって初期化するのがよいと思っています。この DataSource を作る部分を本家が scalikejdbc-hikaricp-xxx みたいなライブラリとして提供するのはありえるかもしれません。scalikejdbc-config に CP の impl ごとの分岐入れるのは微妙かなと。 http://scalikejdbc.org/documentation/connection-pool.html#hikaricp
Kazuhiro Sera
@seratch
Mar 27 2016 14:01
とはいえ、現時点で現実的な選択肢となってる connection pool の実装ってそんなに多くないと思うので scalikejdbc-config である程度特別対応するのも解決の方法がよければ悪くはないとも思います。
Tsuyoshi Yoshizawa
@tsuyoshizawa
Mar 27 2016 14:10
なるほど、了解です。設定ファイルを読み込む処理を各自実装しないといけないのが面倒そうだったので、そのあたりカバーできたらよりいいなと思った程度でした。瀬良さんが提案している解決方法でも改善されるのでよいと思います。
Kazuhiro Sera
@seratch
Mar 27 2016 14:11
最近チェックしてなかったけど Play は今はこういうことになってるんですね。私自身はこれを追加すること自体にあまりモチベーションがないのですが、誰かが移植してくれるならやってもいいかもしれません。 https://www.playframework.com/documentation/2.5.x/SettingsJDBC
すみません、最初の話に戻ってるw 私としては強く反対する気持ちはないけど、積極的に入れたいとまでは思ってなくて、これがどうしても欲しい人が対応してくれてその実装がよさそうなら merge してもよいかもという温度感ですね。
Kazuhiro Sera
@seratch
Mar 27 2016 14:18
Play ユーザに限った話で言えば scalikejdbc-play-dbapi-adapter を使うことにして Play 側の実装にのっかればいいんじゃないかなと思います。
Tsuyoshi Yoshizawa
@tsuyoshizawa
Mar 27 2016 14:20
そうですねー。僕も Play のプロジェクトはそっちに乗っからせてもらってます :smile:
Kazuhiro Sera
@seratch
Mar 27 2016 14:26
ところで 2.4 はいつ出しますかね..
#507 はまあ私の方でテストケース追加するということで merge してもいいけど #423 は gakuzzzz さん..
まあ 4 月中には出そうということで、無理なく。でも、進めましょう。
Manabu Nakamura
@gakuzzzz
Mar 27 2016 14:30
手が回ってなくてすみませぬ……。はい。
ところで parameterExtractor を隠蔽する話、 SQL側だけじゃなくて SQLSyntax#parameters 側でもやったほうがいいですよね?
Kazuhiro Sera
@seratch
Mar 27 2016 14:32
あ、確かに統一したいですね。
Manabu Nakamura
@gakuzzzz
Mar 27 2016 14:32
拝承、その方向で直します。
Kazuhiro Sera
@seratch
Mar 27 2016 14:34
:bow: