From: Junio C Hamano <gitster@pobox.com>
Date: Tue, 28 Oct 2014 19:07:58 GMT
Subject: Re: [PATCH 05/15] refs.c: update rename_ref to use a transaction
Message-ID: <xmqqppdcj9m9.fsf@gitster.dls.corp.google.com>
In-Reply-To: <1413923820-14457-6-git-send-email-sahlberg@google.com>

Ronnie Sahlberg <sahlberg@google.com> writes:

> commit 0295e9cebc41020ee84da275549b164a8770ffba upstream.
>
> Change refs.c to use a single transaction to copy/rename both the refs and
> its reflog. Since we are no longer using rename() to move the reflog file
> we no longer need to disallow rename_ref for refs with a symlink for its
> reflog so we can remove that test from the testsuite.

Do you mean that we used to do a single rename(2) to move the entire
logfile, but now you copy potentially thousands of reflog entries
one by one?

Hmmmm,... is that an improvement?