Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FIX the ticker memory leak #40395

Closed
Dontmindmes opened this issue Jul 24, 2020 · 5 comments
Closed

FIX the ticker memory leak #40395

Dontmindmes opened this issue Jul 24, 2020 · 5 comments

Comments

@Dontmindmes
Copy link

@Dontmindmes Dontmindmes commented Jul 24, 2020

Hey, i made a program that sends requests to my HTTP website using a ticker and everytime the ticker runs my memory goes up gradually by 0.1-0.50 and does not go back down. Memory is not being released could be please be fixed? the application is turning to google chrome

@networkimprov
Copy link

@networkimprov networkimprov commented Jul 24, 2020

Could you fill out the new-issue template, and provide a complete Go program that demonstrates this? That would let us track down a Go bug, or point out any mistakes in your code. Thanks!

@gopherbot add WaitingForInfo

@ianlancetaylor
Copy link
Contributor

@ianlancetaylor ianlancetaylor commented Jul 24, 2020

Note that the docs for time.NewTicker clearly say "Stop the ticker to release associated resources."

@Dontmindmes
Copy link
Author

@Dontmindmes Dontmindmes commented Jul 24, 2020

There is also a very high CPU usage of 13%

@ianlancetaylor
Copy link
Contributor

@ianlancetaylor ianlancetaylor commented Jul 24, 2020

At the moment there is no action we can take here. If you want to help us fix something, please provide a test case that we can run to see the problem. Thanks.

@gopherbot
Copy link

@gopherbot gopherbot commented Aug 25, 2020

Timed out in state WaitingForInfo. Closing.

(I am just a bot, though. Please speak up if this is a mistake or you have the requested information.)

@gopherbot gopherbot closed this Aug 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
5 participants
You can’t perform that action at this time.