git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Nicolas Pitre <nico@cam.org>
To: Brandon Casey <casey@nrlssc.navy.mil>
Cc: Jeff King <peff@peff.net>, Matthieu Moy <Matthieu.Moy@imag.fr>,
	John Dlugosz <JDlugosz@TradeStation.com>,
	git@vger.kernel.org
Subject: Re: dangling commits and blobs: is this normal?
Date: Wed, 22 Apr 2009 16:07:25 -0400 (EDT)	[thread overview]
Message-ID: <alpine.LFD.2.00.0904221601200.6741@xanadu.home> (raw)
In-Reply-To: <I5p8gPPuE_qW2RDhwiqxCWDuMtnuvvgtSkeTkxby6rlj_FKtpERaBA@cipher.nrlssc.navy.mil>

On Wed, 22 Apr 2009, Brandon Casey wrote:

> But isn't git-gc supposed to be the "high-level" command that just does
> the right thing?  It doesn't seem to me to be outside the scope of this
> command to make a decision about trading off speed/io for optimal repo
> layout.  In fact, it does do this already.  The default window, depth and
> compression settings are chosen to be "good enough", not to produce the
> absolute optimum repo.

Exact.

> I'm just pointing out that everything is a trade off.  So I think saying
> something like "gc must optimize for git's performance" is not entirely
> accurate.  We make tradeoffs now.  Other tradeoffs may be helpful.

Git makes tradeoffs for itself.  Trying to optimize by _default_ for 
some random backup system, or any other environmental component not 
involved in git usage, is completely silly.

> Also, don't interpret my comments as me being convinced that a change to
> gc should be made.  It's a trivial patch, but I'm not yet certain one
> way or the other.

Be free to interpret my replies as me being certain of not doing such a 
change.


Nicolas

  parent reply	other threads:[~2009-04-22 20:09 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-21 21:46 dangling commits and blobs: is this normal? John Dlugosz
2009-04-22 15:27 ` Jeff King
2009-04-22 16:53   ` Brandon Casey
2009-04-22 17:39     ` Nicolas Pitre
2009-04-22 18:15       ` Matthieu Moy
2009-04-22 19:08         ` Jeff King
2009-04-22 19:45           ` Brandon Casey
2009-04-22 19:58             ` Jeff King
2009-04-22 20:07             ` Nicolas Pitre [this message]
2009-04-23 11:51           ` Matthieu Moy
2009-04-22 19:14         ` Nicolas Pitre
2009-04-22 19:26       ` Brandon Casey
2009-04-22 20:00         ` Nicolas Pitre
2009-04-22 20:05           ` Jeff King
2009-04-22 20:11             ` Nicolas Pitre
2009-04-23 17:43             ` Geert Bosch
2009-04-23 17:56               ` Shawn O. Pearce
2009-04-23 18:10                 ` Geert Bosch
2009-04-23 18:17                   ` Matthias Andree
2009-04-23 18:51               ` Nicolas Pitre
2009-04-22 20:15   ` John Dlugosz

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=alpine.LFD.2.00.0904221601200.6741@xanadu.home \
    --to=nico@cam.org \
    --cc=JDlugosz@TradeStation.com \
    --cc=Matthieu.Moy@imag.fr \
    --cc=casey@nrlssc.navy.mil \
    --cc=git@vger.kernel.org \
    --cc=peff@peff.net \
    /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).