diff options
| author | Eric Wong (Contractor, The Linux Foundation) <e@80x24.org> | 2018-03-02 20:46:55 +0000 |
|---|---|---|
| committer | Eric Wong (Contractor, The Linux Foundation) <e@80x24.org> | 2018-03-02 20:46:55 +0000 |
| commit | 44ff716b116f10ae42372f9f517356000636e571 (patch) | |
| tree | b69afdd2f8c37c6fec0c7c17e97d9349b6d2341f /lib/PublicInbox/SearchIdx.pm | |
| parent | f80ef6a32b28f54415f79513537f6c9fcd58a244 (diff) | |
| download | public-inbox-44ff716b116f10ae42372f9f517356000636e571.tar.gz | |
'Q' is merely a convention in the Xapian world, and is close enough to unique for practical purposes, so stop using XMID and gain a little more term length as a result.
Diffstat (limited to 'lib/PublicInbox/SearchIdx.pm')
| -rw-r--r-- | lib/PublicInbox/SearchIdx.pm | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/lib/PublicInbox/SearchIdx.pm b/lib/PublicInbox/SearchIdx.pm index 57aed75c..61dc0577 100644 --- a/lib/PublicInbox/SearchIdx.pm +++ b/lib/PublicInbox/SearchIdx.pm @@ -19,7 +19,7 @@ use POSIX qw(strftime); require PublicInbox::Git; use constant { - MAX_MID_SIZE => 244, # max term size - 1 in Xapian + MAX_MID_SIZE => 244, # max term size (Xapian limitation) - length('Q') PERM_UMASK => 0, OLD_PERM_GROUP => 1, OLD_PERM_EVERYBODY => 2, @@ -302,7 +302,7 @@ sub add_message { } $smsg = PublicInbox::SearchMsg->new($mime); my $doc = $smsg->{doc}; - $doc->add_term('XMID' . $mid); + $doc->add_term('Q' . $mid); my $subj = $smsg->subject; my $xpath; @@ -404,7 +404,7 @@ sub remove_message { $mid = mid_clean($mid); eval { - my ($head, $tail) = $self->find_doc_ids('XMID' . $mid); + my ($head, $tail) = $self->find_doc_ids('Q' . $mid); if ($head->equal($tail)) { warn "cannot remove non-existent <$mid>\n"; } @@ -721,7 +721,7 @@ sub create_ghost { my $tid = $self->next_thread_id; my $doc = Search::Xapian::Document->new; - $doc->add_term('XMID' . $mid); + $doc->add_term('Q' . $mid); $doc->add_term('G' . $tid); $doc->add_term('T' . 'ghost'); |