画像探索が遅くなっている(Playwright関連コミットで)

[Nyoho]

(masterを使ってみて気付きました) 5.7.0時点と比べてmasterのビルドがものすごく遅くなっていました。

プロファイラで調べたところ、このコミット a430616#diff-ad508b9b3fad48162f8630146009a8859aaa4ba68d0b1a1eacbdf29d97e4e74a で増えた

review/lib/review/book/image_finder.rb

Line 39 in a430616

if downname == "#{target}#{ext}" || File.absolute_path(downname) == File.absolute_path("#{target}#{ext}")

の、次の部分がすごく遅いようでした。

File.absolute_path(downname) == File.absolute_path("#{target}#{ext}")

この探索アルゴリズムでは downname == "#{target}#{ext}" がほとんどの場合偽になるので、ほとんど後ろの File.absolute_path が評価されてしまって、システムを使い、すごく遅くなっているのではないかと思いました。

実験

今書いている本で計測しました。画像が160個ぐらいあります。
その1個1個に find_path メソッドが数百ミリ秒かかっていました。

1. || File.absolute_path... ありでは rake epub に 106 秒
2. || File.absolute_path... なしでは rake epub に 30 秒

懸念

このコミットがPlaywright Runnerを入れるコミットの一部であり、Playwrightにこの || File.absolute_path が必要なのかも知れません。しかしPlaywrightがよくわからない僕にはわかりません。
(自分のプロジェクトでは File.absolute_path が必要になることはなさそうなのですが)

案

変更案を考えてみました。

1. Playwrightに || File.absolute_path が不必要なのでしたら外す
2. 必要なのでしたら、まずFile.absolute_pathを使わない探索をして(高速)、それでもヒットしなかったらFile.absolute_pathを込めた探索をする実装に変更する

[kmuto]

あら、それは大変。ありがとうございます、確認します!

[Nyoho]

ありがとうございました!