Close `nns` resolver after use #324

New Issue

Closed

opened 2024-02-27 12:57:55 +00:00 by dkirillov · 0 comments

dkirillov commented

2024-02-27 12:57:55 +00:00

Collaborator

Currently we don't close nns resolver. So after app startup we have 68 goroutine but can reduce this count to 58.

We don't leak, but it would be nice to close resources that aren't used anymore.

So at leas we have to add defer nns.Close() here. But probably we have some other places.

Currently we don't close nns resolver. So after app startup we have 68 goroutine but can reduce this count to 58. We don't leak, but it would be nice to close resources that aren't used anymore. So at leas we have to add `defer nns.Close()` [here](https://git.frostfs.info/TrueCloudLab/frostfs-s3-gw/src/commit/391fc9cbe3c95b26bd5dc684a9c09bed0a6e6865/internal/frostfs/util/util.go#L31). But probably we have some other places.