analysis
This commit is contained in:
parent
31e78c8894
commit
f914ee61e3
2 changed files with 42 additions and 0 deletions
|
@ -0,0 +1,19 @@
|
|||
[[!comment format=mdwn
|
||||
username="joey"
|
||||
subject="""comment 1"""
|
||||
date="2024-07-31T14:20:51Z"
|
||||
content="""
|
||||
Note that this does not affect the number of commits made by `addurl` generally
|
||||
eg when adding multiple urls with --batch from the web.
|
||||
|
||||
Also, I don't think that the commits you picked out and showed necessarily
|
||||
correspond to one-another. The state being recorded in the commit in the 1st
|
||||
run is not the same as the state that gets recorded by the two commits in the
|
||||
2nd run. Unless, there is an actual behavior change that eg, leaves the file
|
||||
present in a repository that it was not present in before.
|
||||
|
||||
In the first run the commit shows key
|
||||
MD5E-s5--db87ebcba59a8c9f34b68e713c08a718.dat ends up recorded as present in
|
||||
datalad-archives but not in the local repository. In the second run, the
|
||||
commits show that the same key ends up recorded present in both repositories.
|
||||
"""]]
|
|
@ -0,0 +1,23 @@
|
|||
[[!comment format=mdwn
|
||||
username="joey"
|
||||
subject="""comment 2"""
|
||||
date="2024-07-31T16:06:38Z"
|
||||
content="""
|
||||
Bisected to [[!commit 780367200b14d532f745079dfa09ffaa214d0a84]],
|
||||
"remove dead nodes when loading the cluster log".
|
||||
|
||||
Replacing `loadClusters` with a noop on top of that commit gets the test
|
||||
suite passing again.
|
||||
|
||||
Since nothing in `loadClusters` involves the location log at all, I think
|
||||
this must come down to a difference in when/if git-annex starts reading
|
||||
from the git-annex branch. There could be git-annex commands that didn't
|
||||
used to read from the branch before, that now do. Which might mean merging
|
||||
in other git-annex branches at different points in time than happened
|
||||
before, which I suppose can result in an additional commit.
|
||||
|
||||
Unfortunately, I can't avoid the early `loadClusters` for reasons explained
|
||||
in that commit.
|
||||
|
||||
Anyway, I doubt this will result in a lot of additional commits.
|
||||
"""]]
|
Loading…
Add table
Add a link
Reference in a new issue