git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Junio C Hamano <gitster@pobox.com>
To: Jens Axboe <jens.axboe@oracle.com>
Cc: git@vger.kernel.org, Linus Torvalds <torvalds@linux-foundation.org>
Subject: Re: auto gc again
Date: Wed, 19 Mar 2008 14:27:11 -0700	[thread overview]
Message-ID: <7vd4pq2ymo.fsf@gitster.siamese.dyndns.org> (raw)
In-Reply-To: <20080318180118.GC17940@kernel.dk> (Jens Axboe's message of "Tue, 18 Mar 2008 19:01:18 +0100")

Jens Axboe <jens.axboe@oracle.com> writes:

> But the new magic is really beside the point. Doing this 'for you' is
> extremely annoying behaviour. I often work on my notebook, so disk is
> both slow and battery is precious. I DON'T want gc to run automatically,
> EVER. Not on repos I have had going for ages, not on ones I just cloned.
> Please bury this silly policy and replace it with a printf() telling me
> that I may increase my performance by running git gc. Don't just do it.
> git does not know better.

Well, earlier, git used to be "kick-ass fast, flexible and powerful if you
knew what you are doing, and if you don't, then you are forever lost" type
of a system, and I think early adopters even took pride in saying so.

Being in the scene myself from early on, I certainly sympathise with that
feeling, and sometimes when a newcomer starts making noises about dumbing
git down without understanding implications (e.g. hiding or removing the
index), I have to resist the urge to say "you need to learn certain new
concepts that do not even exist counterparts in earlier crap systems you
are used to.  If you feel you are confused, that's your problem.  Get
enlightened first."  I rarely say that out loud, to be more diplomatic,
though.

But judging from the fact that some kernel folks talking about having a
7GB kernel repository, I think supposedly early adoptors may not really
know what they are doing, and some automation, if done correctly would be
a good thing.

Having said that, I am not sure how the auto gc is triggering for your
(presumably reasonably well maintained) repository that has only small
number of loose objects.  I haven't seen auto-gc annoyance myself (and
git.git is not the only project I have my git experience with), and Linus
also said he hasn't seen breakages.

I think we did have a few patches to the area recently and we should not
rule out the possibility that we broke the criteria "gc --auto" kicks in.

  parent reply	other threads:[~2008-03-19 21:28 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
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 [this message]
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=7vd4pq2ymo.fsf@gitster.siamese.dyndns.org \
    --to=gitster@pobox.com \
    --cc=git@vger.kernel.org \
    --cc=jens.axboe@oracle.com \
    --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).