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

warning: dumping very large path (> 256 MiB); this may run out of memory #602

Closed
bjornfor opened this issue Aug 12, 2015 · 4 comments
Closed

Comments

@bjornfor
Copy link
Contributor

I think it would be useful if the message was changed in two ways:

  • the actual size of the path is printed (replacing the "> 256" part)
  • the path to the path(!) is printed
@edolstra
Copy link
Member

Well, it won't know the size of the path until it has dumped all of it, which might take much longer and cause OOM.

@bjornfor
Copy link
Contributor Author

Doh, I didn't know that. So that message is printed at the time process has copied 256 MiB, not when the copying starts?

But the path to the path is feasible to add, right?

@copumpkin
Copy link
Member

This confused me too, FWIW

@vcunat
Copy link
Member

vcunat commented Jan 8, 2016

I'd personally close this in favor of #358. (I don't have the permissions.)

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

4 participants