アプリがパッチにまみれたら

アプリがパッチにまみれたら Powered�by�Rabbit�2.1.9
アプリがパッチに
まみれたら
TokyuRuby会議11
2017-07-29�(Sat)
伊藤�浩⼀�(@koic)

�

ドリンク
アップス
ポンサー

ティルックラル�第391篇
学ぶ価値のある知識はすべて
学べ学んだ知恵を⽣かして⾏
動せよ

今⽇
の話

rails�
new�以降
はパッチ

パッチいろいろ
機能拡張✓
バグ修正✓
パフォーマンス改善✓
依存Gemのアップグレード✓
リファクタリング✓
機能削除✓

結論

良いパッチにま
みれれば幸い、
悪いパッチにま
みれたら⾟い。

良いパッ
チと悪い
パッチ

良いパッチ
本質的な問題を解決している✓
必要最⼩限でいて拡張の邪魔に
ならないパッチ
✓

悪いパッチ
その場凌ぎで問題解決している✓
本質的な問題解決となっておら
ずしばしば拡張の邪魔になるパ
ッチ
とはいえ問題解決はしているので、
度合いの問題ではある
✓
✓

今⽇の
お題

みんな大好
きモンキー
パッチ

課題

アプリケーションの
ドメインコードと、
フレームワーク/ラ
イブラリの汎用コー
ドが同⼀リポジトリ
に混在する

分け
よう

本来フォーカスす
る問題領域のみア
プリケーションの
コードとしてリポ
ジトリにあるべき

依存する
OSSを含
めた観点

良いパッチ
適切な�upstream�に対するパ
ッチを送って、本質的な問題解
決をする
✓
アプリケーションコードに依存
Gem特有の問題を混ぜない
✓

問題の残るパッチ
とりあえず⼿元のアプリケーシ
ョンが動くワークアラウンド
✓
あの時のあのパッチがあればが
今のアプリケーションに埋もれ
たままになるかも
✓

天使の助⾔と悪魔の囁き
問題の抽象化やupstream向け
のテストを書く⼿間はある
往々にして汎化問題より特化問題の
⽅が簡単
✓
✓
「おまえもモンキーパッチにし
てやろうか」
✓

モンキーパッチの⽌むを得な
い場合
フレームワーク/ライブラリへ
の抽象化が難しい、本当にアプ
リケーション特有の問題
✓
RailsAdmin�での�CSV�ダウン
ロードの�ShiftJIS�変換とか
あれ？この例パッチ送れるのでは...✓
✓

⼈間の葛藤

悪魔の囁き
に敗れた際
の対処事例

bundle�updateへ負債対策
��
bundle�update�に注意みたいな
コメントではなく、動くコード
での警告にしておく。
✓
Gemﬁleへのバージョン固定にす
ると固定されて終わる
✓

Extract�Gem
外部Gemにして開発サイクル
を分ける
✓
アプリケーション固有の問題で
ないので切り出しGemﬁle�1⾏
にする
✓
事例:�https://github.com/
koic/oracle-enhanced-
monky̲patch̲755
✓

Gemﬁle
��
��
Rails�4.2.1でタイムゾーンがズ
レる問題を直すGemのはずが、
Rails�5でタイムスタンプ型が変
わったことへの対応も⼊った
✓
Oracleでのスキーマ移⾏を延命
できる
✓

Railsアップグレードとスキ
ーマ
RDBMSのスキーマの型変更は
重い話になりうる
✓
Railsアップグレードと異なる
タイミングで⾏なえるようにし
たのは悪い判断ではなかった
✓

究極の弱点
activerecord-
oracle̲enhanced-adapter-
monky̲patch̲755
✓
わざとイケてない名前にしたけ
れど、本当にイケてなくてつら
い
全⼒でこの名前を消しにかかる✓
✓

パッチ戦争
からGem
戦争へ

モンキーパッチGemを倒す
Gem
https://github.com/koic/
screamers
✓
oracle-enhanced-
monky̲patch̲755を倒すた
めに誕⽣した
✓
怨念駆動開発✓

screamers:migration
��
��
dateからdatetimeに⼀括変換す
るマイグレーションファイルを
⽣成する
✓

天使界と
悪魔界の
狭間

Forked�Gem
upstreamのmasterでは直っ
ているがリリースされていない
✓
PRは出しているがコミッター
が活動していない
✓

Rails�5.0.1�+�Ruby�2.4
warning:�constant�::Fixnum�
is�deprecated�川が氾濫して
いた
✓
⾒たいログが警告に流されて⾒
えない
✓
警告川の氾濫✓

Rails�5.0.1で警告を鎮まら
せた
��
��
��
rails/railsの5-0-stableからブラ
ンチを切って、必要なコミット
をmasterからcherry-pick
✓
Rails�5.0.2までのあいだ警備し
ていた
✓

反応がない。どうしたら...
��
��
パッチはある。反応はない。✓
Gemの選定は大事✓

天使の助
⾔が勝っ
たとき

Upstreamに投げる
Oracleに詳しいわけではない
ので、DBAに⾒てもらえる
✓
新しい知識を得ることが出来る✓
パッチ投げていたら、コミッタ
ーに推薦してもらえた
✓

⼩さな仕事、大きな成功
パッチの送り先はアプリケーシ
ョンに限らない
✓
依存先のupstreamにも出来る
ことからコツコツと
✓
少しずつ出来ることが増えて⾏
くので、カルマを積み重ねて⾏
きましょう
✓

Empfohlen