Re: Slow pushes on 'pu' - even when up-to-date..
- From
Jeff King <peff@peff.net>
- Date
- Oct 4, 2016, 16:21 UTC
- Message-ID
- <20161004162102.rwofudnx3g3fsyul@sigill.intra.peff.net>
- In-Reply-To
- <xmqq8tu4w0i2.fsf@gitster.mtv.corp.google.com>
On Tue, Oct 04, 2016 at 09:19:01AM -0700, Junio C Hamano wrote:
Show 15 quoted lines
> Jeff King <peff@peff.net> writes: > > > As I argued in [1], I think it's not just "this must be cheaper" but > > "this must not be enabled if submodules are not in use at all". Most > > repositories don't have submodules enabled at all, so anything that > > cause any extra traversal, even of a portion of the history, is going to > > be a net negative for a lot of people. > > > > I think the only sane default is going to be some kind of heuristic that > > says "submodules are probably in use". > > Why should we even have a default different from today's? If most > repositories don't have submodules enabled at all, we can just let > those working with submodules enabled to toggle their configuration > and that is an very easy to understand solution, no?
You will not see any complaint from me on that. I was taking for granted that the current default is inconvenient to submodule users, but I don't have any experience myself.
-Peff