From: david@lang.hm
To: "Shawn O. Pearce" <spearce@spearce.org>
Cc: Junio C Hamano <junkio@cox.net>, Dana How <danahow@gmail.com>,
Git Mailing List <git@vger.kernel.org>
Subject: Re: [PATCH] Prevent megablobs from gunking up git packs
Date: Thu, 24 May 2007 10:23:39 -0700 (PDT) [thread overview]
Message-ID: <Pine.LNX.4.64.0705241020450.21766@asgard.lang.hm> (raw)
In-Reply-To: <20070524071235.GL28023@spearce.org>
On Thu, 24 May 2007, Shawn O. Pearce wrote:
> Now #3 is actually really important here. Don't forget that we
> *just* disabled the fancy "new loose object format". It doesn't
> exist. We can read the packfile-like loose objects, but we cannot
> write them anymore. So lets say we explode a megablob into a loose
> object, and its 800 MiB by itself. Now we have to send that object
> to a client. Yes, that's right, we must *RECOMPRESS* 800 MiB for
> no reason. Not the best choice. Maybe we shouldn't have deleted
> that packfile formatted loose object writer...
when did the object store get changed so that loose objects aren't
compressed?
if the problem is that the codepath for fetching does an uncompress
followed by a compress then it would seem that this is a fairly easy
problem to fix (how hard would it be to add the headers around the
compressed object to make it look to the receiver like it's a pack with
only one thing in it)
David Lang
next prev parent reply other threads:[~2007-05-24 17:23 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-05-22 6:14 [PATCH] Prevent megablobs from gunking up git packs Dana How
2007-05-22 6:30 ` Shawn O. Pearce
2007-05-22 7:33 ` Dana How
2007-05-22 6:52 ` Junio C Hamano
2007-05-22 8:00 ` Dana How
2007-05-22 11:05 ` Jakub Narebski
2007-05-22 16:59 ` Dana How
2007-05-22 23:44 ` Jakub Narebski
2007-05-23 0:28 ` Junio C Hamano
2007-05-23 1:58 ` Nicolas Pitre
2007-05-22 17:38 ` Nicolas Pitre
2007-05-22 18:07 ` Dana How
2007-05-23 22:08 ` Junio C Hamano
2007-05-23 23:55 ` Dana How
2007-05-24 1:44 ` Junio C Hamano
2007-05-24 7:12 ` Shawn O. Pearce
2007-05-24 9:38 ` Johannes Schindelin
2007-05-24 17:23 ` david [this message]
2007-05-24 17:29 ` Johannes Schindelin
2007-05-25 0:55 ` Shawn O. Pearce
2007-05-24 20:43 ` Geert Bosch
2007-05-24 23:29 ` Dana How
2007-05-25 2:06 ` Shawn O. Pearce
2007-05-25 5:44 ` Nicolas Pitre
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=Pine.LNX.4.64.0705241020450.21766@asgard.lang.hm \
--to=david@lang.hm \
--cc=danahow@gmail.com \
--cc=git@vger.kernel.org \
--cc=junkio@cox.net \
--cc=spearce@spearce.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).