Skip to content

remote: optimize rm/prune ref deletion

Jens Lindström git 2 versions In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest3 patches 3 repliesIn review
  2. v12 patches 6 repliesSuperseded

Changes since previous version from the cover letter

* Additionally change the order that 'remote rm' does things so that it removes the remote configuration as the last step and only if the other steps succeeded.

* Change the packed-refs repacking patch to repack before deleting refs instead of after. This makes the patch simpler, since delete_ref() no longer needs to be told not to repack.

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3remote rm: delete remote configuration as the lastbuiltin/remote.c+6 -41 reply
2/3remote: repack packed-refs once when deleting multiple refsbuiltin/remote.c refs.c refs.h+21 -32 replies
3/3remote prune: optimize "dangling symref" check/warningbuiltin/remote.c refs.c refs.h+25 -20 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v2 yet.

Read the v2 thread

Cover letter [PATCH v2 0/3] remote: optimize rm/prune ref deletion

Changes since previous version:

* Additionally change the order that 'remote rm' does things so that it removes the remote configuration as the last step and only if the other steps succeeded.

* Change the packed-refs repacking patch to repack before deleting refs instead of after. This makes the patch simpler, since delete_ref() no longer needs to be told not to repack.

Jens Lindstrom (3):
remote rm: delete remote configuration as the last step
remote: repack packed-refs once when deleting multiple refs remote prune: optimize "dangling symref" check/warning

 builtin/remote.c | 37 ++++++++++++++++++++++++++++++-------
 refs.c           | 21 +++++++++++++++++++--
 refs.h           |  3 +++
 3 files changed, 52 insertions(+), 9 deletions(-)

--
1.9.1