[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [jgit-dev] How to deal with LargeObjectException in RevWalk.parseCommit?
- From: Shawn Pearce <spearce@xxxxxxxxxxx>
- Date: Wed, 4 Apr 2012 11:31:15 -0700
- Delivered-to: email@example.com
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=spearce.org; s=google; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc:content-type; bh=ZA/zJ8xfsgv2HjkQC0BTWCOu0g6egJcZYCVD8N28zVA=; b=V6wQ1NziEwhaeGN8rK4DW3qxhxwanNOLlmk64/5MUC4DLI6mWltfkuq+PuNiNamDbM m+QObSFj180jIg2YrgRxCYQpW1nEe3FQ+TGX5ZNEuy3vgPI7WC8O9WYHv9p9GXtZBOWv /F3NFGkgYAF96duhu67sBc2nwDwPCjpksXPKg=
On Wed, Apr 4, 2012 at 11:24, Marc Strapetz <marc.strapetz@xxxxxxxxxxx> wrote:
>> We could make this a repository level configuration setting, with the
>> default being the current 5 MiB. If users need it bigger, they could
>> set a larger limit in .git/config or ~/.gitconfig.
> OK. In addition I'd still add a setter to change the default limit, e.g.
> SmartGit usually runs with -Xmx256M, so I'd probably have 128M as
> default here. The few remaining users which use their pagefile.sys for
> the commit message will have to change their repository configuration.
> If there are no objections, I'll prepare a corresponding patch.
It probably makes sense to connect this with the WindowCacheConfig. Or
maybe we just use core.streamFileThreshold here?
Oh, now I remember. I wanted to make sure we could process most
commits, so I just hard coded this to 5 MiB. But we could do the
larger of 5 MiB or core.streamFileThreshold.