Thread (1 message) 1 message, 1 author, 2022-09-24

Re: [PATCH 4/4] diff-lib: parallelize run_diff_files for submodules

From: Junio C Hamano <hidden>
Date: 2022-09-24 20:17:10

Ævar Arnfjörð Bjarmason [off-list ref] writes:
On Thu, Sep 22 2022, Calvin Wan wrote:
quoted
+status.parallelSubmodules::
+	When linkgit:git-status[1] is run in a superproject with
+	submodules, a status subprocess is spawned for every submodule.
+	This option specifies the number of submodule status subprocesses
+	to run in parallel. If unset, it defaults to 1.
Why do we default to 1, instead of e.g. grep.threads defaulting to the
"cores available"?
I would imagine we would want to be able to say:

 - I do not trust the parallel mode yet, just use the single process
   method that we have always been using.

 - I do not know how many cores I have, just use a reasonable
   default parallelism.

 - I want to use N processes because I know better than auto-scaling
   based on num_cpus.

And the value of 1 would be a reasonable way to express the first
one.

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help