Re: [PATCH v3 0/8] Maintenance II: prefetch, loose-objects, incremental-repack tasks

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Derrick,

> On Aug 25, 2020, at 20:36, Derrick Stolee via GitGitGadget <gitgitgadget@xxxxxxxxx> wrote:
> 

...

> 
> Updates since v2
> ================
> 
> * Dropped "fetch: optionally allow disabling FETCH_HEAD update"
> 
> 
> * A lot of fallout from the change in the option parsing in v3 of
>   Maintenance II.
> 
> 
> * Dropped the "verify, and delete and rewrite on failure" logic from the
>   incremental-repack task. This might be added again later after it can be
>   tested more thoroughly.

Perhaps I missed some conversations related to this change but
why was this verify-rewrite strategy dropped?

Was the problem such strategy were created to solve is now no longer a concern?

I feel like it would be much better to add it in and then remove it using a separated commit?
That way we can follow the reasoning behind these decisions via commit message.

> 

...

> 
> -- 
> gitgitgadget

Thanks,
Son Luong.



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux