RE: [PATCH] git-svn: make batch mode optional for git-cat-file

2015-10-11 Thread Victor Leschuk
ds, Victor From: Eric Wong [normalper...@yhbt.net] Sent: Wednesday, September 23, 2015 12:22 PM To: Victor Leschuk Cc: Junio C Hamano; git@vger.kernel.org Subject: Re: [PATCH] git-svn: make batch mode optional for git-cat-file Victor Leschuk wrote: > Hello Eric, thank

Re: [PATCH] git-svn: make batch mode optional for git-cat-file

2015-09-23 Thread Eric Wong
Victor Leschuk wrote: > Hello Eric, thanks for looking into it. > > >> git-cat-file has outgrown the parent perl process several times > >> (git-cat-file - ~3-4Gb, perl - 400Mb). > > > Ugh, that sucks. > > Even the 400Mb size of Perl annoys me greatly and I'd work > > on fixing it if I had more

RE: [PATCH] git-svn: make batch mode optional for git-cat-file

2015-09-23 Thread Victor Leschuk
ide git-cat-file, I think we could give it a try, I can prepare a patch and run some tests. -- Best Regards, Victor From: Eric Wong [normalper...@yhbt.net] Sent: Tuesday, September 22, 2015 5:35 PM To: Victor Leschuk Cc: Junio C Hamano; git@vger.kernel.org Subject: Re: [PATCH] git-svn: ma

Re: [PATCH] git-svn: make batch mode optional for git-cat-file

2015-09-22 Thread Eric Wong
Eric Wong wrote: > Victor Leschuk wrote: > > The thing is that git-cat-file keeps growing during work when running > > in "batch" mode. See the figure attached: it is for cloning a rather > > small repo (1 hour to clone about ~14000 revisions). However the clone > > of a large repo (~28 revis

Re: [PATCH] git-svn: make batch mode optional for git-cat-file

2015-09-22 Thread Eric Wong
Victor Leschuk wrote: > The thing is that git-cat-file keeps growing during work when running > in "batch" mode. See the figure attached: it is for cloning a rather > small repo (1 hour to clone about ~14000 revisions). However the clone > of a large repo (~28 revisions) took about 2 weeks and

Re: [PATCH] git-svn: make batch mode optional for git-cat-file

2015-09-22 Thread Junio C Hamano
Victor Leschuk writes: > We already do have some of these: 'no-metadata', 'no-checkout', > no-auth-cache'. So I was just following the existing convention. Do > you think we need to change it and stick with > --catch-file-batch=1/--cat-file-batch=0 ? Inventing a new --cat-file-batch=[0|1] is not

RE: [PATCH] git-svn: make batch mode optional for git-cat-file

2015-09-22 Thread Victor Leschuk
: git@vger.kernel.org Subject: RE: [PATCH] git-svn: make batch mode optional for git-cat-file Hello Junio, thanks for your review. First of all I'd like to apologize for sending the patch without description. Actually I was in a hurry and sent it by accident: I planned to edit the mail b

RE: [PATCH] git-svn: make batch mode optional for git-cat-file

2015-09-21 Thread Victor Leschuk
__ From: Junio C Hamano [jch2...@gmail.com] On Behalf Of Junio C Hamano [gits...@pobox.com] Sent: Monday, September 21, 2015 11:25 AM To: Victor Leschuk Cc: git@vger.kernel.org; Victor Leschuk Subject: Re: [PATCH] git-svn: make batch mode optional for git-cat-file Victor Leschu

Re: [PATCH] git-svn: make batch mode optional for git-cat-file

2015-09-21 Thread Junio C Hamano
Victor Leschuk writes: > Signed-off-by: Victor Leschuk > --- Before the S-o-b line is a good place to explain why this is a good change to have. Please use it. > git-svn.perl | 1 + > perl/Git.pm | 41 - > 2 files changed, 41 insertions(+), 1 deletio

[PATCH] git-svn: make batch mode optional for git-cat-file

2015-09-21 Thread Victor Leschuk
Signed-off-by: Victor Leschuk --- git-svn.perl | 1 + perl/Git.pm | 41 - 2 files changed, 41 insertions(+), 1 deletion(-) diff --git a/git-svn.perl b/git-svn.perl index 36f7240..b793c26 100755 --- a/git-svn.perl +++ b/git-svn.perl @@ -139,6 +139,7 @@ m