summaryrefslogtreecommitdiff
path: root/doc/bugs/added_branches_makes___39__git_annex_unused__39___slow
diff options
context:
space:
mode:
authorGravatar http://joeyh.name/ <http://joeyh.name/@web>2013-08-11 12:48:47 +0000
committerGravatar admin <admin@branchable.com>2013-08-11 12:48:47 +0000
commit0c1868c455d1d1d8cb07670576d36cfd64978180 (patch)
treee7ce69a5421e952647b1625f880b217a1966c875 /doc/bugs/added_branches_makes___39__git_annex_unused__39___slow
parent0129dbad772f9d6800d8476dd34c25a7378f21a6 (diff)
Added a comment
Diffstat (limited to 'doc/bugs/added_branches_makes___39__git_annex_unused__39___slow')
-rw-r--r--doc/bugs/added_branches_makes___39__git_annex_unused__39___slow/comment_4_a50b43c15d2650df90f0fa1ced47f532._comment10
1 files changed, 10 insertions, 0 deletions
diff --git a/doc/bugs/added_branches_makes___39__git_annex_unused__39___slow/comment_4_a50b43c15d2650df90f0fa1ced47f532._comment b/doc/bugs/added_branches_makes___39__git_annex_unused__39___slow/comment_4_a50b43c15d2650df90f0fa1ced47f532._comment
new file mode 100644
index 000000000..3de85407e
--- /dev/null
+++ b/doc/bugs/added_branches_makes___39__git_annex_unused__39___slow/comment_4_a50b43c15d2650df90f0fa1ced47f532._comment
@@ -0,0 +1,10 @@
+[[!comment format=mdwn
+ username="http://joeyh.name/"
+ ip="2001:4978:f:21a::2"
+ subject="comment 4"
+ date="2013-08-11T12:48:47Z"
+ content="""
+I think that could work. It would probably tend to use more memory than the current method, but only a small constant multiplier more. And unused is already the one command that necessarily needs to hold information about the whole repository in memory.
+
+Note that git cat-file is only needed when dealing with branches other than the current working tree. In that special case, it can, and AFAIK does have the optimisation of looking directly at the symlink target instead. Your method may turn out to be both slower and use more memory in that case. It may make sense to special case handling of that case and keep the current code paths there (most of the necessary code for it is used by other stuff anyway).
+"""]]