about summary refs log tree commit homepage
path: root/lib/PublicInbox/SearchIdx.pm
diff options
authorEric Wong <e@80x24.org>2020-12-25 10:21:12 +0000
committerEric Wong <e@80x24.org>2020-12-26 06:22:56 +0000
commitfb4dd7fdeeed8478cda9b7e63e56564da8cbdacf (patch)
treee50ce0cadd50c5210c0b4f53239e8db605a529bc /lib/PublicInbox/SearchIdx.pm
parent14e606423429d6121c295c2bc0599fe1bf66b07c (diff)
We'll count the number of log changes (regardless of index or
unindex) and only attach inboxes to ExtSearchIdx objects when
they get new work.  We'll also reduce lock bouncing and only
update external indices after all per-inbox indexing is done.

This also updates existing v2 indexing/unindexing callers
to be more consistent and ensures unindex log entries update
per-inbox last commit information.
Diffstat (limited to 'lib/PublicInbox/SearchIdx.pm')
1 files changed, 2 insertions, 0 deletions
diff --git a/lib/PublicInbox/SearchIdx.pm b/lib/PublicInbox/SearchIdx.pm
index c8e309fc..b3361e05 100644
--- a/lib/PublicInbox/SearchIdx.pm
+++ b/lib/PublicInbox/SearchIdx.pm
@@ -615,6 +615,7 @@ sub index_both { # git->cat_async callback
         $smsg->{num} = index_mm($self, $eml, $oid, $sync) or
                 die "E: could not generate NNTP article number for $oid";
         add_message($self, $eml, $smsg, $sync);
+        ++$self->{nidx};
         my $cur_cmt = $sync->{cur_cmt} // die 'BUG: {cur_cmt} missing';
         ${$sync->{latest_cmt}} = $cur_cmt;
@@ -629,6 +630,7 @@ sub unindex_both { # git->cat_async callback
         if (defined(my $cur_cmt = $sync->{cur_cmt})) {
                 ${$sync->{latest_cmt}} = $cur_cmt;
+        ++$self->{nidx};
 sub with_umask {