Go言語で定数として扱いたいmapを毎回アロケートさせて性能劣化した話 | フューチャー技術ブログ

はじめに失敗談をテーマにした連載の3本目です。 TIG DXユニットの原です。21年度4月に新卒で入社し、2年目となります。 参加しているプロジェクトで、数百万件のデータを処理する機能を担当したことがありました。 本記事はその際の失敗と、その失敗から得た経験を共有するため、執筆しました。 内容のサマリ 本来フィールドで宣言すべき定数的に扱いたい変数を、関数内で毎回生成しreturnしてしまったので呼び出す度に毎回アロケートが発生し性能劣化してしまった Benchmark Test、Profiling、Escape Analysisでどのような挙動になってしまっていたか調べてみた 内容本記事では、まずどのような状況であったかを説明し、どのような順番で問題を解決したかの順で説明します。 主にGoのテストとProfilingに関した内容です。 Goのテストについての関連記事として、Goのテストに

[mattn]

map が理由ではなく return してるから当然ヒープですよね。「Go言語ではmap型は基本Heapメモリにエスケープするため、使う際には注意が必要であるになります。」

[turanukimaru]

毎回作ってるのが問題で map とかヒープとかではないような。新コードで return map してるのも"他の個所でマスタ情報の変更ができないように"が解決できてないので func MapGet(i int) string {return m[i]} を作るのがスジでわ…？

[odz]

見た瞬間、そりゃ馬鹿でかいオブジェクトを毎回生成してりゃ遅いだろという感想しかない。問題箇所特定に手間取るのは理解出来るが、以降は細かい分析いらんだろ。

[ene0kcal]

以前の職場でやはり無意味なアロケートで性能が超ダウンしてたので指摘し、改善方法提示したのに、前に検討したとかいってやらないことを即断されたのみたので、本件は製品版を直したのか気になる。

[havanap]

ひとに説明するのに便利そう

[snowcrush]

"他の個所でマスタ情報の変更ができないようにGetterを作りました" そもそもGetterでimmutableには出来ないね。あとそもそもimmutableにする必要もないような？使われているのはテストでしょ。

[mano-junki]

失敗談をテーマにした連載

[asuka0801]

“Getterが呼ばれるたびに、mapデータを作っていたわけです” map作ってreturnしてるならそらそうよ。

[onesplat]

まだ右も左もわからない新卒に無理やり技術記事を書かせることにより生じる会社の技術レベルに対するレピュテーションリスク。

[Fluss_kawa]

ヒープとか関係ない気がする。関数が呼ばれるたびに構築されてりゃそりゃ遅いでしょと。コンパイラが最適化してインライン化でもすれば別かもしれんが。

[nmcli]

案外ミスらないと分からんもの