From: Jeff King Date: Fri, 08 Sep 2017 07:42:57 GMT Subject: Re: [PATCH 00/10] Implement transactions for the packed ref store Message-ID: <20170908074257.tmdyj4wbij35fylq@sigill.intra.peff.net> In-Reply-To: On Tue, Aug 29, 2017 at 10:20:24AM +0200, Michael Haggerty wrote: > This should be the second-to-last patch series in the quest for > mmapped packed-refs. I just gave this a careful read, and it looks pretty good to me. I had a few questions, but no show-stoppers. I'll admit to glossing a little bit over the conversions in patch 3 and 4, after seeing that they're basically adaptations of the code that goes away in patch 9. -Peff