mysql 5.7 でdeadlockが発生 #630

[seasoftjapan]

• トランザクション分離レベルを意識して、無駄な SQL 実行を避ける。mysql 5.7 でdeadlockが発生 #630 (comment)
• 実行コストが幾らか低い傾向の exists() を使う。mysql 5.7 でdeadlockが発生 #630 (comment)
• SC_DB_DBFactory のテストが無さそうなので、今回追加したメソッドの他、ごく基本的な部分を実装する。
• 削除件数を返さない不具合を修正。
• アノテーションを補完。

[codecov]

Codecov Report

Attention: Patch coverage is 75.00000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 49.94%. Comparing base (55353cf) to head (24855f2).
Report is 4 commits behind head on master.

Files with missing lines	Patch %	Lines
data/class/SC_Query.php	25.00%	3 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1124      +/-   ##
==========================================
- Coverage   50.62%   49.94%   -0.69%     
==========================================
  Files          80       82       +2     
  Lines       10263    10480     +217     
==========================================
+ Hits         5196     5234      +38     
- Misses       5067     5246     +179     

Flag	Coverage Δ
tests	49.94% <75.00%> (-0.69%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[nanasess]

LGTM

[nobuhiko]

ちなみにmysql5.7で単純に1万回実行してパフォーマンス比較したら若干劣化してましたよ

[seasoftjapan]

@nobuhiko MySQL は、若干そうなるはずです。シナリオ次第で、PostgrteSQL で実際に削除されるケースでは、概ね2倍のパフォーマンスでした。