Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Let "git svn" run "git gc --auto" occasionally
Let "git svn" run "git gc --auto" every 1000 imported commits to
reduce the number of loose objects.

To handle the common use case of frequent imports, where each
invocation typically fetches much less than 1000 commits, also run gc
unconditionally at the end of the import.

"1000" is the same number that was used by default when we called
git-repack. It isn't necessarily still the best choice.

Signed-off-by: Karl Hasselström <kha@treskal.com>
Acked-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
kwiberg authored and gitster committed Feb 3, 2008
1 parent af788a6 commit 94bc914
Showing 1 changed file with 12 additions and 0 deletions.
12 changes: 12 additions & 0 deletions git-svn.perl
Expand Up @@ -1247,6 +1247,8 @@ package Git::SVN;
use File::Copy qw/copy/;
use IPC::Open3;

my ($_gc_nr, $_gc_period);

# properties that we do not log:
my %SKIP_PROP;
BEGIN {
Expand Down Expand Up @@ -1407,6 +1409,7 @@ sub read_all_remotes {
}

sub init_vars {
$_gc_nr = $_gc_period = 1000;
if (defined $_repack || defined $_repack_flags) {
warn "Repack options are obsolete; they have no effect.\n";

This comment has been minimized.

Copy link
@john-peterson

john-peterson Jan 5, 2013

Why not connect these flags to gc_period so that git gc can be disabled for git svn clone without disabling git gc? This would be especially appropriated when the file system is slow (f.e. magnetic storage).

}
Expand Down Expand Up @@ -2095,6 +2098,10 @@ sub restore_commit_header_env {
}
}

sub gc {
command_noisy('gc', '--auto');
};

sub do_git_commit {
my ($self, $log_entry) = @_;
my $lr = $self->last_rev;
Expand Down Expand Up @@ -2148,6 +2155,10 @@ sub do_git_commit {
0, $self->svm_uuid);
}
print " = $commit ($self->{ref_id})\n";
if (--$_gc_nr == 0) {
$_gc_nr = $_gc_period;
gc();
}
return $commit;
}

Expand Down Expand Up @@ -3975,6 +3986,7 @@ sub gs_fetch_loop_common {
$max += $inc;
$max = $head if ($max > $head);
}
Git::SVN::gc();
}

sub match_globs {
Expand Down

0 comments on commit 94bc914

Please sign in to comment.