我有以下代码:
func Call(ctx context.Context, payload Payload) (Response, error) {
req, err := http.NewRequest(...) // Some code that creates request from payload
ctx, cancel = context.withTimeout(ctx, time.Duration(3) * time.Second)
defer cancel()
return http.DefaultClient.Do(req)
}
如果我不把defer cancel()
放进去会发生什么?go vet
警告说
应该调用而不是丢弃context.WithTimeout返回的cancel函数,以避免上下文泄漏
上下文将如何泄露,这将产生什么影响?谢谢
2条答案
按热度按时间qoefvg9y1#
如果取消上下文失败,goroutine that WithCancel or WithTimeout created将无限期地保留在内存中(直到程序关闭),从而导致内存泄漏。如果你经常这样做,你的记忆力会显著膨胀。最佳做法是在调用
WithCancel()
或WithTimeout()
后立即使用defer cancel()
lokaqttq2#
如果使用
WithCancel
,goroutine 将无限期地保存在内存中。但是,如果您使用WithDeadline
或WithTimeout
而不调用cancel,则 goroutine 将只保留到计时器到期。但这仍然不是最佳实践,最好在处理完资源后立即调用
cancel
。