golangでメモリの解放を明示的に行う必要があるか

Question

起きている事象

• 以下のプログラムをメモリ容量の少ないサーバ（ec2:t2.micro、ラズベリーパイ:4GBなど）で動作させると3hほどで接続ができなくなったりフリーズし再起動が必要になる

ざっくり仕様

• ビットコインの情報を取得（チャネルで表現）
• 取得後にいろいろ加工してテーブルへ保存

現状までの調査結果

• 取得する処理自体では問題はなさそう（起きている事象は発生しない）
• いろいろ加工してテーブルへ保存する処理で何か問題が発生している模様

テーブル保存時に毎回やっている処理（エラーハンドリングは省略してます）

• 時間変換系処理

func (t *Ticker) TruncateDateTime(duration time.Duration) time.Time {
	return t.DateTime().Truncate(duration)
}

func (t *Ticker) DateTime() time.Time {
	dateTime, err := time.Parse(time.RFC3339, t.Timestamp)
	return dateTime
}

• レコードを形成する処理

func NewCandle(productCode string, duration time.Duration, timeDate time.Time, open, close, high, low, volume float64) *candleInfraStruct {
	return &candleInfraStruct{
		ProductCode: productCode,
		Duration:    duration,
		Time:        timeDate,
		Open:        open,
		Close:       close,
		High:        high,
		Low:         low,
		Volume:      volume,
	}
}

• 保存する処理

candle := candle.NewCandle(productCode, duration, ticker.TruncateDateTime(duration),
	price, price, price, price, ticker.Volume)
	candle.Insert()

• 更新する処理

currentCandle.Save()

現状やっている対策

• 保存が終わったタイミングでruntime.GC()をやって検証中。

調査方法などご指摘いただければ幸いです。

Comments

[toast-uz]

CPU稼働率が100%なのに、メモリを疑っているのは何故ですか？
問題の性格から、OSや実行環境（ec2なのか物理サーバなのかとか）の情報が必要だと思います。

[sanezane]

■CPUについて
メモリ不足のエラーが発生した際のエラーハンドリングが悪いことが原因でした。今回の事象とは直接的な関係はありませんでしたので事象について修正しました。

[toast-uz]

ulimitでメモリ使用量に制限をかけてみて、様子をみてはいかがでしょうか。

[nobonobo]

情報不足のためなんとも回答しにくいです。提示コードに問題があるようには見えませんし、せめて繰り返してる処理の一巡くらいは切り出して欲しいところ。

Answer 1

質問のお題に関しては「メモリの解放を明示的に行う必要」は「GCがあるため不要です」が回答になります。

ただ、3hほどでフリーズするという現象はメモリーリークやリソースリークが疑わしいですね。
ただGCがあればこれらのような現象が絶対に観測されない事はなく、それは起こり得ます。

• gorouttineの停止漏れ
• OSリソース解放漏れ（閉じる必要のある類）
• 解放する速度よりもメモリ利用量の増加速度の方が早い

などなどの状況が予想されますが、これより具体的な回答は実装の詳細が見えないとなんとも回答できません。