From mboxrd@z Thu Jan 1 00:00:00 1970 From: Brian Downing Subject: [PATCH 1/5] Don't try to delta if target is much smaller than source Date: Wed, 11 Jul 2007 22:14:14 -0500 Message-ID: <11842100581148-git-send-email-bdowning@lavos.net> References: <11842100581060-git-send-email-bdowning@lavos.net> Cc: Junio C Hamano , Brian Downing To: git@vger.kernel.org X-From: git-owner@vger.kernel.org Thu Jul 12 05:14:25 2007 Return-path: Envelope-to: gcvg-git@gmane.org Received: from vger.kernel.org ([209.132.176.167]) by lo.gmane.org with esmtp (Exim 4.50) id 1I8p8K-0005sZ-Dg for gcvg-git@gmane.org; Thu, 12 Jul 2007 05:14:24 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1765360AbXGLDOS (ORCPT ); Wed, 11 Jul 2007 23:14:18 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S934829AbXGLDOS (ORCPT ); Wed, 11 Jul 2007 23:14:18 -0400 Received: from 74-134-246-243.dhcp.insightbb.com ([74.134.246.243]:57749 "EHLO silvara" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1763968AbXGLDOR (ORCPT ); Wed, 11 Jul 2007 23:14:17 -0400 Received: by silvara (Postfix, from userid 1000) id 1FFE352135; Wed, 11 Jul 2007 22:14:18 -0500 (CDT) X-Mailer: git-send-email 1.5.2.GIT In-Reply-To: <11842100581060-git-send-email-bdowning@lavos.net> Sender: git-owner@vger.kernel.org Precedence: bulk X-Mailing-List: git@vger.kernel.org Archived-At: Add a new try_delta heuristic: Don't bother trying to make a delta if the target object size is much smaller (currently 1/32) than the source, as it's very likely not going to get a match. Even if it does, you will have to read at least 32x the size of the new file to reassemble it, which isn't such a good deal. This leads to a considerable performance improvement when deltifying a mix of small and large files with a very large window, because you don't have to wait for the large files to percolate out of the window before things start going fast again. Signed-off-by: Brian Downing --- builtin-pack-objects.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/builtin-pack-objects.c b/builtin-pack-objects.c index 54b9d26..132ce96 100644 --- a/builtin-pack-objects.c +++ b/builtin-pack-objects.c @@ -1342,6 +1342,8 @@ static int try_delta(struct unpacked *trg, struct unpacked *src, sizediff = src_size < trg_size ? trg_size - src_size : 0; if (sizediff >= max_size) return 0; + if (trg_size < src_size / 32) + return 0; /* Load data if not already done */ if (!trg->data) { -- 1.5.2.GIT