slick-joda-mapperの件

slick-joda-mapper-issue.md

slick-joda-mapperの動作がおかしそう

概要

• tototoshi/slick-joda-mapper#30 でCalenderを使う実装がはいった
• 具体的なところはmysql-connector-javaの実装による疑惑があるが、setTimestampでCalendarを渡すと実際にDBに対して発行されるINSERTなどのクエリーの時間の表現が変化すると思われる

  • Calendarオブジェクトを使用すると、ドライバはカスタム・タイムゾーンを考慮してタイムスタンプを計算できます。

  • https://docs.oracle.com/javase/jp/8/docs/api/java/sql/PreparedStatement.html#setTimestamp-int-java.sql.Timestamp-java.util.Calendar-
• ただ👇下記のようにタイムゾーンを変更した時間をDBに入れるテストを追加するとCalendarを利用するとおかしなことが起きてしまう
  • https://github.com/tototoshi/slick-joda-mapper/compare/master...y-yu:slick-joda-mapper:xuwei-k-setDateTime-with-test
• 吉田さんのPR tototoshi/slick-joda-mapper#77 では、互換性を考慮してCalenderを利用しない（tototoshi/slick-joda-mapper#30 が入る前の実装）に戻せるよう方法が追加された
  • この実装だと上記のテストは全てSuccessする

どういうときに変なことになるのか？

• まず、現在は次のようにグローバルな設定をやっている

  Locale.setDefault(Locale.JAPAN)
  val tz = TimeZone.getTimeZone("Asia/Tokyo")
  TimeZone.setDefault(tz)
  DateTimeZone.setDefault(DateTimeZone.forID(tz.getID))

  • https://github.com/tototoshi/slick-joda-mapper/blob/2f0c8fde79ecc5b5dfe7abf1a21bdd20226dd761/src/test/scala/com/github/tototoshi/slick/JodaSupportSpec.scala#L99-L102
• そしてテストの結果がこのようになる

  [info] - should be the same in/out joda-time zoned Asia/Tokyo
  [info] - should be the same in/out joda-time zoned UTC *** FAILED ***
  [info]   1354806000000 was not equal to 1354838400000 (JodaSupportSpec.scala:252)
  [info] - should be the same in/out joda-time zoned Europe/London *** FAILED ***
  [info]   1354806000000 was not equal to 1354838400000 (JodaSupportSpec.scala:252)
  [info] - should be the same in/out joda-time zoned America/New_York *** FAILED ***
  [info]   1354806000000 was not equal to 1354856400000 (JodaSupportSpec.scala:252)
  

• 保存されるDateTimeのゾーンがjava.util.TimeZoneのグローバル設定と異なるときに発生していると考えられる
  • たとえば下記のようにjava.util.TimeZoneをUTCに設定すると、UTCとEurope/Londonは通って他が落ちるというようになる

  val tz = TimeZone.getTimeZone("UTC")
  TimeZone.setDefault(tz)

互換性について

• 吉田さんのPRは互換性に最大限配慮されていて、tototoshi/slick-joda-mapper#77 をマージしても基本現在の利用者に影響はない
  • PRのコメントにもあるように、最初も自分はそれで「なおってないじゃないか🤔」と勘違いしてました……😇
• 吉田さんのPRがマージされたあとの修正バージョンを使いたい人は、下記のように明示的に使う必要がある

  - val jodaSupport = com.github.tototoshi.slick.MySQLJodaSupport
  + val jodaSupport = new com.github.tototoshi.slick.GenericJodaSupport(MySQLProfile, _ => None)