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

Memory usage is very large #2

Open
EverCurse opened this issue Oct 19, 2018 · 2 comments
Open

Memory usage is very large #2

EverCurse opened this issue Oct 19, 2018 · 2 comments

Comments

@EverCurse
Copy link

When I use this tool to query a lot of urls,urls will put the result to groutines,finally i find
resp, err := reqPool.PooledClient.Do(req) will create a a lot of groutines,and memory is fastly Growing。

@phayes
Copy link

phayes commented Feb 26, 2020

It looks like it's leaking goroutines:

goroutine profile: total 12101
8340 @ 0x432df0 0x406cfb 0x406cd1 0x406ab5 0x935303 0xa34f3e 0xa34f22 0x460901
#	0x935302	github.com/ewohltman/pool.(*PClient).DoPool+0x132	/src/vendor/github.com/ewohltman/pool/pool.go:92
#	0xa34f3d	github.com/ewohltman/pool.(*PClient).Do+0x28d		/src/vendor/github.com/ewohltman/pool/pool.go:79

3576 @ 0x432df0 0x406cfb 0x406cd1 0x406ab5 0x935303 0xa341da 0xa341be 0x460901
#	0x935302	github.com/ewohltman/pool.(*PClient).DoPool+0x132	/src/vendor/github.com/ewohltman/pool/pool.go:92
#	0xa341d9	github.com/ewohltman/pool.(*PClient).Do+0x679		/src/vendor/github.com/ewohltman/pool/pool.go:79

@phayes
Copy link

phayes commented Feb 26, 2020

It actually could be that it's not handling backpressure at all, and just spinning up goroutines until it OOMs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants