SELECT ... FOR UPDATE同士でデッドロックさせる - かみぽわーる

最近SELECT ... FOR UPDATEでデッドロックする話を何度かしたので。 前職のときにUPDATE同士がデッドロックしてたときに、SELECT ... FOR UPDATEで排他ロックを取ってからUPDATEしてデッドロックを防ぎますってPRをレビューしてたときのことで、複数レコードの排他ロックは一瞬ですべてのレコードのロックを取れるわけではなく、ロックを取る順番が揃っていないと簡単にデッドロックしますよという話です。 https://gist.github.com/kamipo/0bb4e37d58ba18a8cefb8aa02f778231 # frozen_string_literal: true require "mysql2" def client Mysql2::Client.new( host: "localhost", username: "root", dat

[kamipo]

blogged

[hdampty7]

ロックをかけるときは意図的にロックテーブル作ってPKでロックをとるようにするというバッドノウハウがある。DBが複雑になって来るとデッドロックは追跡が難しく深刻な問題。

[oakbow]

複数テーブルと読み違えてたら複数レコードか。なるほど走査の順番が違うとデッドロック起きるな〜。こういう同時実行制御の問題は知らないと回避しづらい。不具合発生しても原因まで自力でたどり着くのほぼ無理

[nakag0711]

諦めてデッドロックしたら再実行しよう。最悪手動で

[electrolite]

完全に防ぐ方法はない。ロックエスカレーションもあるから順番だけでも解決しない。RDBMSのデッドロック検知に任せて起きたらアプリを直す。RDB誕生時からの定理。しかしRDBは長命レガシーだな。また一周回った。

[t_f_m]

"ロックを取る順番が一意になるようにクエリや実行計画を揃えるというのがこの手の問題に対する一般的な対処法になります"

[mizchi]

いい話 “かくいう僕も眼の良さを活かして気合いで対処してる”

[ichigen3]

これってファントムリードを回避するためにギャップロックがかかるのが原因だから、トランザクション分離レベルを、デフォルトのRepeatable ReadではなくRead Commitedしようって話な気がするけども。

[koyancya]

これは知らないと一生ハマりそうだ...... -> “ロックを取る順番って見えづらいので問題に気づきづらいですよね”

[tmatsuu]

そうか。情報処理技術者試験でロックはテーブル単位で順序を守ると学んでたのでこういう状況が発生することを把握できてなかった。ロックはテーブル単位とは限らないというのを認識する必要がある。

[t-tanaka]

SQLは，代表的な宣言型プログラミング言語。手続き型言語に比べ，テキシコーのいうところの「頭の中で手順をたどる」が極端に難しい。人類には早いのではないか？というきがしてならない。

[sawarabi0130]

車のキーをかばんに入れて、そのかばんを車に入れてとじ込みするとデッドロックが発生する

[p1ass]

知らなかった

[buhoho]

普段脳みそ停止して扱ってたけどDBってデッドロックするんかーい！！全然考えてなかったわ。今まで遭遇しなかったのは運がいいだけっぽいな…

[Eiichiro]

PKロック使ってました。確かにバッドノウハウというか、古の手法感はある。分離レベルはDB2のRCだったかな。あとは処理を全Insertにしてロックを取得しない方法もあるけど、本番レベルで試したことはないなぁ。

[mp0liiu]

インデックスの捜査順が違うとデッドロック発生するとか考えたこともなかった・・・ / 助かりました

[moto_motosaka]

知らなかった。ただ、for updateは複数レコードに対して、あんまり実行しないようにしてる気がする。ブコメ見ると、これには分離レベルの話も関わってくるのね

[rjge]

“ロックを取る順番が一意になるようにクエリや実行計画を揃えるというのがこの手の問題に対する一般的な対処法になります。 ロックを取る順番って見えづらいので問題に気づきづらい”

[quick_past]

完全に同時にロックを外すなんてのは書き方の上での方便であって、実際には順番に外すしかできないんだから、当然の結果ではあると思う

[minoton]

dbms(&オプション)によっても挙動が違うしねえ

[tettekete37564]

そんなこと起きるんだ。

[kenzy_n]

嵌ったことがある方にはしんみりとくる

[versatile]

最近 dead lock はタイムアウトするので、それでいいじゃんって思い込んでた老害

[yamadar]

show processlist; kill $pid;

[kibitaki]

意図的にデッドロックを起こして〇〇する、って中二病方面ですごい好まれそうな技だけどあまり採用例ないね。

[Kesin]

SELECT FOR UPDATEするときはWHEREで使うPKを事前にソートしてからSQLを組み立てていた

[oranie]

かくいう僕も眼の良さを活かして気合いで対処してるので

[ken39arg]

どうしても解決できなかったデッドロックか？と思ったけど少し違いそうだった。排他制御は難しい。それが面白いけど

[khigashigashi]

複数レコードのロックは一瞬で同時に起きるわけではなく、順番に起きて途中の状態が存在するので、ロックを取る順番が一意になるようにクエリや実行計画を揃える

[gfx]

"ロックを取る順番が一意になるようにクエリや実行計画を揃えるというのがこの手の問題に対する一般的な対処法"

[cartman0]

データベーススペシャリスト的なやつの問題にないのかしら

[waihasaruya]

学び

[yo_waka]

"ロックを取る順番が一意になるようにクエリや実行計画を揃えるというのがこの手の問題に対する一般的な対処法になります"