a40ec5e03e
When hashing the files, the entire list of shas was read strictly. That was entirely unnecessary, since there's a cleanup action run after they're consumed.
41 lines
1.9 KiB
Markdown
41 lines
1.9 KiB
Markdown
For the record, `git annex add` has had a series of memory leaks.
|
|
Mostly these are minor -- until you need to check in a few
|
|
million files in a single operation.
|
|
|
|
If this happens to you, git-annex will run out of memory and stop.
|
|
(Generally well before your system runs out of memory, since it has some
|
|
built-in ulimits.) You can recover by just re-running the `git annex add`
|
|
-- it will automatically pick up where it left off.
|
|
|
|
A history of the leaks:
|
|
|
|
* Originally, `git annex add` remembered all the files
|
|
it had added, and fed them to git at the end. Of course
|
|
that made its memory use grow, so it was fixed to periodically
|
|
flush its buffer. Affected versions: before 0.20110417
|
|
|
|
* Something called a "lazy state monad" caused "thunks" to build
|
|
up and memory to leak. Also affected other git annex commands
|
|
than `add`. Adding files using a SHA* backend hit the worst.
|
|
Fixed in versions afer 3.20120123.
|
|
|
|
* A strange GHC bug seemed to be responsible for another leak.
|
|
(In particular, a child process was forked. All the child did
|
|
was read filenames from one pipe and shove them reformatted out
|
|
another pipe. For some reason, it steadily grew in size.)
|
|
Code was rewritten in a way that happens to avoid that leak.
|
|
Apparently fixed in versions afer 3.20120123, but this one is not
|
|
well understood.
|
|
|
|
* Committing journal files turned out to have another memory leak.
|
|
After adding a lot of files ran out of memory, this left the journal
|
|
behind and could affect other git-anne commands. Fixed in versions afer
|
|
3.20120123.
|
|
|
|
* (Note that `git ls-files --others`, which is used to find files to add,
|
|
also uses surpsisingly large amounts
|
|
of memory when you have a lot of files. It buffers
|
|
the entire list, so it can compare it with the files in the index,
|
|
before outputting anything.
|
|
This is Not Our Problem, but I'm sure the git developers
|
|
would appreciate a patch that fixes it.)
|