Re: [RFC 0/16] Introduce index file format version 5
- From
Thomas Gummerer <t.gummerer@gmail.com>
- Date
- Aug 3, 2012, 12:46 UTC
- Message-ID
- <20120803124632.GD3226@tgummerer.unibz.it>
- In-Reply-To
- <CACsJy8AQUwb8noutwdcN-6Piyo1rrj7_EUMjrXB-83J9AtM6kQ@mail.gmail.com>
On 08/03, Nguyen Thai Ngoc Duy wrote:
Show 8 quoted lines
> On Thu, Aug 2, 2012 at 6:01 PM, Thomas Gummerer <t.gummerer@gmail.com> wrote: > > Series of patches to introduce the index version 5 file format. This > > series does not include any fancy stuff like partial loading or partial > > writing yet, though it's possible to do that with the new format. > > I applied the series on top of master. I had to manually resolve > 09/16. You may want to rebase the series on master for the reroll > (less work for Junio) and remove trailing whitespaces in the patches.
Thanks, I'll do that for the reroll.
Show 9 quoted lines
> All tests passed (with v5 by default (*), I notice it now), which is > wonderful. I'll have a closer look on the following days. Thank you > for working on this. > > (*) while it's good to run tests with v5 by default. I'm not sure if > we should make it by default in the next git release that comes with > v5. For one thing it'll stop older gits from using shared repos. And > we're not sure whether v5 introduces significant overhead in common > use cases.
I didn't intent to make index v5 the format, I just oversaw the hunk that was changing it. I completely agree that v5 should not be default yet to avoid any surprises, also for users of jgit, libgit2 etc.