diff options
author | Eric Wong <e@80x24.org> | 2019-06-14 17:42:57 +0000 |
---|---|---|
committer | Eric Wong <e@80x24.org> | 2019-06-14 21:56:40 +0000 |
commit | 097cca4c583a3fe51945406041d1d88c864689a5 (patch) | |
tree | f744148fa87ba051b583989b9b67baccedc1818d /lib/PublicInbox/SearchIdx.pm | |
parent | 3f5370d63630701d0f62f57c15e08d566c35db42 (diff) | |
download | public-inbox-097cca4c583a3fe51945406041d1d88c864689a5.tar.gz |
search*: rename {partition} => {shard}
Another step towards keeping our internal data structures consistent with Xapian naming.
Diffstat (limited to 'lib/PublicInbox/SearchIdx.pm')
-rw-r--r-- | lib/PublicInbox/SearchIdx.pm | 17 |
1 files changed, 9 insertions, 8 deletions
diff --git a/lib/PublicInbox/SearchIdx.pm b/lib/PublicInbox/SearchIdx.pm index a088ce75..58b23375 100644 --- a/lib/PublicInbox/SearchIdx.pm +++ b/lib/PublicInbox/SearchIdx.pm @@ -29,7 +29,7 @@ use constant { my $xapianlevels = qr/\A(?:full|medium)\z/; sub new { - my ($class, $ibx, $creat, $part) = @_; + my ($class, $ibx, $creat, $shard) = @_; ref $ibx or die "BUG: expected PublicInbox::Inbox object: $ibx"; my $levels = qr/\A(?:full|medium|basic)\z/; my $mainrepo = $ibx->{mainrepo}; @@ -62,9 +62,9 @@ sub new { my $dir = $self->xdir; $self->{over} = PublicInbox::OverIdx->new("$dir/over.sqlite3"); } elsif ($version == 2) { - defined $part or die "partition is required for v2\n"; - # partition is a number - $self->{partition} = $part; + defined $shard or die "shard is required for v2\n"; + # shard is a number + $self->{shard} = $shard; $self->{lock_path} = undef; } else { die "unsupported inbox version=$version\n"; @@ -102,8 +102,8 @@ sub _xdb_acquire { $self->lock_acquire; # don't create empty Xapian directories if we don't need Xapian - my $is_part = defined($self->{partition}); - if (!$is_part || ($is_part && need_xapian($self))) { + my $is_shard = defined($self->{shard}); + if (!$is_shard || ($is_shard && need_xapian($self))) { File::Path::mkpath($dir); } } @@ -824,9 +824,10 @@ sub commit_txn_lazy { $self->{-inbox}->with_umask(sub { if (my $xdb = $self->{xdb}) { - # store 'indexlevel=medium' in v2 part=0 and v1 (only part) + # store 'indexlevel=medium' in v2 shard=0 and + # v1 (only one shard) # This metadata is read by Admin::detect_indexlevel: - if (!$self->{partition} # undef or 0, not >0 + if (!$self->{shard} # undef or 0, not >0 && $self->{indexlevel} eq 'medium') { $xdb->set_metadata('indexlevel', 'medium'); } |