lerna-util-sequence 改善

[tksugimoto]

• SequenceFactoryWorker
  • Initialize が不要
  • ⚠️ SequenceStore.ReservationFailed の handling 漏れ
  • Command は NoSerializationVerificationNeeded を継承したほうが良い
  • reservationAmount = 1 での採番で無限ループのリスク #44
    • a821301 予約失敗時の番号飛び
• SequenceStore
  • OpenSession が不要
  • final case object ReservationFailed extends RuntimeException の RuntimeException が不要
  • SequenceStoreSpec

    • "障害が発生しても継続して予約できる" ignore { // FIXME: typed 化で同じ仕組みではtestできなくなった

      • Typed/lerna util sequence #29 (comment)
  • state 整理

    • なるほど。session がない notReady と session がある notReady があるんですね。

      • Typed/lerna util sequence #29 (comment) Typed/lerna util sequence by tksugimoto · Pull Request #29 · lerna-stack/lerna-app-library
• settings

  • [IMO] 設定値の不備をアプリ起動時に検知できるように読み込みはできるだけ早い段階で行ったほうが良いと思います。SequenceFactoryConfig 内にこの設定値の読み込みを実装することはできないでしょうか ？

    • 現状 SequenceFactoryCassandraConfig の中も def が多く、実行時エラーが発生しやすい構造なので、別PRでまとめて対策としたいと思います。

    • Typed/lerna util sequence #29 (comment) Typed/lerna util sequence by tksugimoto · Pull Request #29 · lerna-stack/lerna-app-library