From: Jens Axboe <jens.axboe@oracle.com>
To: Nicolas Pitre <nico@cam.org>
Cc: Linus Torvalds <torvalds@linux-foundation.org>, git@vger.kernel.org
Subject: Re: auto gc again
Date: Thu, 20 Mar 2008 08:40:57 +0100 [thread overview]
Message-ID: <20080320074057.GH17940@kernel.dk> (raw)
In-Reply-To: <alpine.LFD.1.00.0803191856290.2947@xanadu.home>
On Wed, Mar 19 2008, Nicolas Pitre wrote:
> On Wed, 19 Mar 2008, Jens Axboe wrote:
>
> > On Wed, Mar 19 2008, Nicolas Pitre wrote:
> > > On Tue, 18 Mar 2008, Jens Axboe wrote:
> > >
> > > > But freshly pulled repo, git auto gc is enabled. And that is my main
> > > > annoyance, I just don't think that type of policy should be in there.
> > >
> > > Just do this once:
> > >
> > > git config --global gc.auto 0
> > > git config --global gc.autopacklimit 0
> > >
> > > and be happy.
> >
> > You don't get it. I did gc.auto 0. And know some other limit crops up, I
> > have to do gc.autopacklimit 0. I have LOTS of git trees. On many
> > machines. It's just annoying, period.
>
> As suggested, gc.auto = 0 should probably be made to disable it
> entirely, regardless of any other parameters that might exist.
Yes, agree.
> > > > Print the warning, include info on how to run git gc or even how to turn
> > > > it on automatically. But I'll bet you that most users will NOT want auto
> > > > gc. Ever.
> > >
> > > Unfortunately, the harshest complaints about this whole issue were the
> > > opposite.
> >
> > I just don't buy that, I have more faith in users.
>
> We also did in the past... even for a long period...
>
> Alas, it is the users who made us (and actually made Linus, who was the
> last to resist) change our minds.
OK, that's at least reassuring :-)
> > If they come around and complain it's slow, heck you told them it
> > would be.
>
> But they don't. They just presume that Git is crap and move on.
That's pretty sad, I like to have high hopes for users.
> > But it's not a big deal, I'll just carry a local patch that disables
> > this crap and forget the whole deal. I just worry that if this is where
> > git 'usability' is heading, it wont be a good thing in the long run.
>
> I wish the majority of users was thinking like you. I, too, have some
> conceptual problems with this auto gc things. With the experience
> we've gathered, the current state appears to be the
> lesser of all evils though.
Alright, I must bow down to empirical evidence... The conceptual policy
problem is indeed what is bothering me so much, even more so than the
actual gc running on my machine.
gc.auto covering everything is good enough for me, GIT_GC_AUTO
environment variable would be better because of the way that I work. But
I can get by knowing that the gc.auto thing will at least only bite me
once per tree. And perhaps just wrap git clone in one of my scripts
that'll then do the gc.auto thing automatically.
--
Jens Axboe
next prev parent reply other threads:[~2008-03-20 7:41 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-03-18 18:01 auto gc again Jens Axboe
2008-03-18 18:14 ` Linus Torvalds
2008-03-18 18:19 ` Jens Axboe
2008-03-18 18:24 ` Jens Axboe
2008-03-18 18:33 ` Linus Torvalds
2008-03-18 18:39 ` Jens Axboe
2008-03-19 20:22 ` Johannes Schindelin
2008-03-19 21:14 ` Jens Axboe
2008-03-19 21:44 ` Johannes Schindelin
2008-03-20 6:00 ` Jens Axboe
2008-03-19 20:37 ` Nicolas Pitre
2008-03-19 21:17 ` Jens Axboe
2008-03-19 23:05 ` Nicolas Pitre
2008-03-20 7:40 ` Jens Axboe [this message]
2008-03-20 7:55 ` Junio C Hamano
2008-03-20 17:31 ` Jens Axboe
2008-03-19 21:27 ` Brandon Casey
2008-03-19 21:53 ` [PATCH] builtin-gc.c: allow disabling all auto-gc'ing by assigning 0 to gc.auto Brandon Casey
2008-03-20 7:08 ` Teemu Likonen
2008-03-19 22:56 ` auto gc again Nicolas Pitre
2008-03-20 6:01 ` Jens Axboe
2008-03-19 21:27 ` Junio C Hamano
2008-03-19 21:52 ` Linus Torvalds
2008-03-19 22:28 ` Junio C Hamano
2008-03-19 23:16 ` Nicolas Pitre
2008-03-19 23:25 ` Junio C Hamano
2008-03-20 3:13 ` Nicolas Pitre
2008-03-20 4:09 ` Junio C Hamano
2008-03-20 4:40 ` Nicolas Pitre
2008-03-20 4:49 ` Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20080320074057.GH17940@kernel.dk \
--to=jens.axboe@oracle.com \
--cc=git@vger.kernel.org \
--cc=nico@cam.org \
--cc=torvalds@linux-foundation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).