プログラムを約3200倍高速化して、社内業務のボトルネックを解消したお話 - Tabelog Tech Blog

はじめに この記事は 食べログアドベントカレンダー2024 の16日目の記事です🎄 こんにちは。食べログ開発本部ウェブ開発1部 システム運用改善チーム所属の @4palaceです。 今回は、私の所属するシステム運用改善チームで、とある社内業務の処理パフォーマンスを改善した事例を紹介します。 この事例では、10日間かかっていた処理を、少しの改修で10分未満に短縮しました。 改修量としては小さくとも、大きなパフォーマンス改善を実現でき、運用業務の効率化につながりました。 個人的に興味深い例でしたので、ここで共有させていただきます。 問題の発見 月次処理が月内に終わらない！ きっかけはカスタマーサポートチームからの相談でした。 「月次の新店舗の登録チェック業務で行われているマッチング処理が遅すぎて、後続の業務も含めて月内に終わらなくなってきている。どうにかできませんか？」 そもそもこの業務とは

[taguch1]

出来立てほやほやの新プロダクト以外であまり最適化の余地がありすぎるとアーキや仕様のレビューが心配になる。

[door-s-dev]

元が悪すぎるとめちゃくちゃ改善したように見えるやーつ。4〜5日の時点でおかしいなってならなかったのが不思議

[yorkfield]

小規模なら問題なく動くし、大規模でも時間かかるだけで動作自体はするから、本格的に問題になるまで放置は珍しい話でもない。手間の割に改善効果が大きいからさっさとやれよと言いたくなる気持ちは分かるけど。

[gomer-pyle]

創業後のカオスな時期に実装され、ただ単にイケてなかったかつ触られることもなかった処理を直すとこのようなことが起きる。

[napsucks]

これまで無駄に10日も待ってたのかと思うとちょっと気の毒

[mohri]

たしかにほかのコメントにあるように「どうしてこれまで改善されてなかったのか」がきになる記事だった

[YassLab]

“10日かかっていた処理が10分未満で完了 / わずか20行程度の修正で爆速になる / 開発を高速化するには、問題・真因のいち早い特定が不可欠”

[daira4000]

そもそも遅過ぎませんか

[semimaru]

どうしてそうなるまで放っておいたの感

[ntstn]

よっぽどの規模でない限りは初っ端からカリカリにチューンしないので、最後まで残った部分だったのかな？

[turanukimaru]

私だったら「自動テストが終わらなくて開発の邪魔になるから速度改善させて欲しい」って言ってしまうと思う。テストできるのが良いコードであり負荷試験においても同じ。

[natu3kan]

手を入れるのが大変そうだからそのままって技術的負債はあるよな。

[soybeancucumber]

そこまで改善の余地を残し続けたってことだろ。。。

[tasukuchan]

“処理”

[fashi]

ORMに任せているとたまにすごい無駄な処理入ることある

[magi00]

むしろ食べログレベルでどうして今まで放置されていたかを共有してほしいやつ。最初の4~5日程度の段階ですでに問題視されそうだけど

[ducktoon]

んなもん相談される前に改善しろってちょっと思った

[kanetann]

.pluck(:id)したのかな