Re: [PATCH v4 0/4] Changed path filter hash fix and version bump

From
Jonathan Tan <hidden>
Date
2023-06-22 22:28:01
Reply to
Derrick Stolee, 2023-06-21
Derrick Stolee [off-list ref] writes:
On 6/21/2023 8:19 AM, Taylor Blau wrote:
quoted
On Tue, Jun 20, 2023 at 09:43:46AM -0400, Derrick Stolee wrote:
quoted
quoted
 commitGraph.changedPathsVersion: Which version should we _write_
 when writing a new commit-graph? Defaults to '1' but will default
 to '2' in the next major verion, then '1' will no longer be an
 accepted value in the version after that.
I am not sure if there's a situation where we'd ever want to not write
the newer versions when starting a new commit-graph (or chain) from
scratch.
I'd rather have the choice to start writing the new filter mode be
made by config rather than a change to the Git binary. Makes for a
more gradual rollout to be sure there aren't issues with the new
version.

So please keep the configuration value, but have it indicate the
mode used when writing filters.

Thanks,
-Stolee
It looks like we can't avoid writing both versions (we need to write
version 1 so that we can reuse existing Bloom filters when writing, if
the repo has version 1 Bloom filters) so a config that tells us which to
write sounds doable. I'll take a look.
Thread (7 messages) 7 messages, 4 authors, last reply 2023-06-23
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help