Gormが本番テーブルの数億件のデータを消そうとした話 - keroxpのScrapbox

MySQLの場合、--safe-updatesオプションを利用することでこういった不慮のUPDATE/DELETEを防げるようです

[takc923]

db.Deleteの引数がSQLのwhereの条件だと考えたら良し悪しは置いておいて意味は分かる。あともうすぐリリースされるらしいv2ではデフォルトでwhere句無しのupdate/deleteはエラーになる。これはマトモな対応じゃないのかな。

[atria]

記事にもあるけどこれと全く同じissueがあがったのが約4年前。修正要望も多い中v2まで対応しないという方針。個人的には対応には疑問符だが、OSSである以上運用はメンテナ次第だし色々考えさせられる

[mihyaeru21]

gormを本番で使ってる人は全員これを有効にすべき https://godoc.org/github.com/jinzhu/gorm#DB.BlockGlobalUpdate

[ledsun]

いやいや、いくらなんでも狙ったレコード（だけ）を削除しているかはテストするでしょ・・・

[fai_fx]

枯れていないというのは、こういう事なんだと思う。

[Kil]

今年12月のQiitaアドベントカレンダー「本番環境でやらかしちゃった話」にエントリーし損ねましたね……

[gabill]

一番の原因はGo言語の(かつての)パッケージ管理の仕組みにあると思う。「同じリポジトリ階層で公開してるなら、破壊的な変更は永久に入れてはならない」みたいな仕組みだった。

[p1ass]

これめっちゃ怖いな

[everybodyelse]

gormはこういうお気持ちを察して使わないと簡単に死ぬので、お勧めしない。

[snowcrush]

GoのORM、だいたい出来が悪いし、そもそもRuby/Railsみたいにランタイムリフレクション駆使してゴリゴリに動的生成する言語じゃないよ（一応Reflectionあるけど）

[igagurimk2]

ORMなんだからさ、ユーザーに見せるインターフェイスはオブジェクトの文法に即してなきゃいかんのよな。

[paulownia]

何それ怖い

[ikaro1192]

yabai

[tmatsuu]

こわい。--safe-updates知らなかった。わいわい。

[takaesu]

ORMでのDELETEの条件が吹っ飛ぶ

[tyru]

ヒエッ

[tkmkg8m]

空 slice 渡したときは何も削除しないでほしいというのはわかる。ユニットテストをしっかり書く習慣があると、 slice の長さが0、1、2あたりでテスト書いて気づけそうな気がする。

[miya2000]

Go-rm

[softstone]

これ大本はSQLの問題だよね。SQL書いてるときはEnter周辺のキーをブラインドタッチしたくないもん。

[cloverstudioceo]

ORM自体最近使ってないなぁ。。。

[namelaw]

あやうくGoneになりかけたね

[jgoamakf]

でもまあ、プログラムで生SQL組み立ててもUPDATEからWHEREが無くなったりしますし……

[stealthinu]

これはおっかなくてちびる。ほんと「たまたま」のおかげでデータ消えなくてよかったな。

[FlowerLounge]

どんなテストケースでテストしたんだろ

[wataken44]

3個のスライスでDelete→3個消える、2個のスライスでDelete→2個消える、1個のスライスでDelete→1個消える、という状況から0個のスライスでDelete→全部消える、は予想できないっすね…(エラーになる or 何もしないでほしい)

[marmot1123]

うわーこわ。

[iww]

『弊社ではこの問題が起きたその日の時点ですべてのGormを使っているコードベースを凍結し、Gormの削除を決定した』

[yatmsu]

これはこわい

[inmysoul]

OSS最高！（無料最高！）ってよだれ垂らしながら使ってりゃそういう罠にもぶち当たるわなw

[uxoru]

”どんなSQLのクエリが生成されるのか分からないORMやDBライブラリは使うべきではない”

[cocoasynn]

ありえないのはまあそうなんだけど OSS だしなー…そこまで辛辣に書く必要もないんじゃとは思う

[klim0824]

"どんなSQLのクエリが生成されるのか分からないORMやDBライブラリは使うべきではない"

[invent]

"どんなSQLのクエリが生成されるのか分からないORMやDBライブラリは使うべきではない"

[nakag0711]

なるほど、条件がORされるなら空の時はなにも削除されないのが自然か。でも全件削除も必要だし、やっぱりこういう仕様にならざるを得ないかも

[rryu]

引数の内容によって発動する機能が変わる系のメソッドあるある。指定されたオブジェクト郡の削除と条件指定の削除の機能が同居しているのでオブジェクト郡が空だと無条件削除の方が発動するという。

[ken39arg]

あるあるではあるけど、ORMありきというのはどうなのかなあ。俺はgoでORMは使ってない。クエリビルダーやscanner系は使うけど

[tmtms]

これはこわい。そもそも db.Delete() って API がイケてない気がする。Go はさっぱりわかんないけど / “だって空だぞ？ 空の配列渡したら全部削除になる意味がわからんだろ”