]> Sergey Matveev's repositories - public-inbox.git/blob - lib/PublicInbox/V2Writable.pm
e1c9a393a36f0fa12b3a04e804984f6981c03223
[public-inbox.git] / lib / PublicInbox / V2Writable.pm
1 # Copyright (C) 2018-2020 all contributors <meta@public-inbox.org>
2 # License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
3
4 # This interface wraps and mimics PublicInbox::Import
5 # Used to write to V2 inboxes (see L<public-inbox-v2-format(5)>).
6 package PublicInbox::V2Writable;
7 use strict;
8 use v5.10.1;
9 use parent qw(PublicInbox::Lock);
10 use PublicInbox::SearchIdxShard;
11 use PublicInbox::IdxStack;
12 use PublicInbox::Eml;
13 use PublicInbox::Git;
14 use PublicInbox::Import;
15 use PublicInbox::MID qw(mids references);
16 use PublicInbox::ContentHash qw(content_hash content_digest);
17 use PublicInbox::InboxWritable;
18 use PublicInbox::OverIdx;
19 use PublicInbox::Msgmap;
20 use PublicInbox::Spawn qw(spawn popen_rd);
21 use PublicInbox::SearchIdx qw(log2stack crlf_adjust is_ancestor check_size);
22 use IO::Handle; # ->autoflush
23 use File::Temp qw(tempfile);
24
25 my $OID = qr/[a-f0-9]{40,}/;
26 # an estimate of the post-packed size to the raw uncompressed size
27 my $PACKING_FACTOR = 0.4;
28
29 # SATA storage lags behind what CPUs are capable of, so relying on
30 # nproc(1) can be misleading and having extra Xapian shards is a
31 # waste of FDs and space.  It can also lead to excessive IO latency
32 # and slow things down.  Users on NVME or other fast storage can
33 # use the NPROC env or switches in our script/public-inbox-* programs
34 # to increase Xapian shards
35 our $NPROC_MAX_DEFAULT = 4;
36
37 sub detect_nproc () {
38         for my $nproc (qw(nproc gnproc)) { # GNU coreutils nproc
39                 `$nproc 2>/dev/null` =~ /^(\d+)$/ and return $1;
40         }
41
42         # getconf(1) is POSIX, but *NPROCESSORS* vars are not
43         for (qw(_NPROCESSORS_ONLN NPROCESSORS_ONLN)) {
44                 `getconf $_ 2>/dev/null` =~ /^(\d+)$/ and return $1;
45         }
46
47         # should we bother with `sysctl hw.ncpu`?  Those only give
48         # us total processor count, not online processor count.
49         undef
50 }
51
52 sub nproc_shards ($) {
53         my ($creat_opt) = @_;
54         my $n = $creat_opt->{nproc} if ref($creat_opt) eq 'HASH';
55         $n //= $ENV{NPROC};
56         if (!$n) {
57                 # assume 2 cores if not detectable or zero
58                 state $NPROC_DETECTED = detect_nproc() || 2;
59                 $n = $NPROC_DETECTED;
60                 $n = $NPROC_MAX_DEFAULT if $n > $NPROC_MAX_DEFAULT;
61         }
62
63         # subtract for the main process and git-fast-import
64         $n -= 1;
65         $n < 1 ? 1 : $n;
66 }
67
68 sub count_shards ($) {
69         my ($self) = @_;
70         my $n = 0;
71         my $xpfx = $self->{xpfx};
72
73         # always load existing shards in case core count changes:
74         # Also, shard count may change while -watch is running
75         # due to "xcpdb --reshard"
76         if (-d $xpfx) {
77                 my $XapianDatabase;
78                 foreach my $shard (<$xpfx/*>) {
79                         -d $shard && $shard =~ m!/[0-9]+\z! or next;
80                         $XapianDatabase //= do {
81                                 require PublicInbox::Search;
82                                 PublicInbox::Search::load_xapian();
83                                 $PublicInbox::Search::X{Database};
84                         };
85                         eval {
86                                 $XapianDatabase->new($shard)->close;
87                                 $n++;
88                         };
89                 }
90         }
91         $n;
92 }
93
94 sub new {
95         # $creat may be any true value, or 0/undef.  A hashref is true,
96         # and $creat->{nproc} may be set to an integer
97         my ($class, $v2ibx, $creat) = @_;
98         $v2ibx = PublicInbox::InboxWritable->new($v2ibx);
99         my $dir = $v2ibx->assert_usable_dir;
100         unless (-d $dir) {
101                 if ($creat) {
102                         require File::Path;
103                         File::Path::mkpath($dir);
104                 } else {
105                         die "$dir does not exist\n";
106                 }
107         }
108         $v2ibx->umask_prepare;
109
110         my $xpfx = "$dir/xap" . PublicInbox::Search::SCHEMA_VERSION;
111         my $self = {
112                 ibx => $v2ibx,
113                 im => undef, #  PublicInbox::Import
114                 parallel => 1,
115                 transact_bytes => 0,
116                 total_bytes => 0,
117                 current_info => '',
118                 xpfx => $xpfx,
119                 over => PublicInbox::OverIdx->new("$xpfx/over.sqlite3"),
120                 lock_path => "$dir/inbox.lock",
121                 # limit each git repo (epoch) to 1GB or so
122                 rotate_bytes => int((1024 * 1024 * 1024) / $PACKING_FACTOR),
123                 last_commit => [], # git epoch -> commit
124         };
125         $self->{over}->{-no_sync} = 1 if $v2ibx->{-no_sync};
126         $self->{shards} = count_shards($self) || nproc_shards($creat);
127         bless $self, $class;
128 }
129
130 # public (for now?)
131 sub init_inbox {
132         my ($self, $shards, $skip_epoch, $skip_artnum) = @_;
133         if (defined $shards) {
134                 $self->{parallel} = 0 if $shards == 0;
135                 $self->{shards} = $shards if $shards > 0;
136         }
137         $self->idx_init;
138         $self->{mm}->skip_artnum($skip_artnum) if defined $skip_artnum;
139         my $epoch_max = -1;
140         git_dir_latest($self, \$epoch_max);
141         if (defined $skip_epoch && $epoch_max == -1) {
142                 $epoch_max = $skip_epoch;
143         }
144         $self->git_init($epoch_max >= 0 ? $epoch_max : 0);
145         $self->done;
146 }
147
148 # returns undef on duplicate or spam
149 # mimics Import::add and wraps it for v2
150 sub add {
151         my ($self, $eml, $check_cb) = @_;
152         $self->{ibx}->with_umask(\&_add, $self, $eml, $check_cb);
153 }
154
155 # indexes a message, returns true if checkpointing is needed
156 sub do_idx ($$$$) {
157         my ($self, $msgref, $mime, $smsg) = @_;
158         $smsg->{bytes} = $smsg->{raw_bytes} + crlf_adjust($$msgref);
159         $self->{over}->add_overview($mime, $smsg);
160         my $idx = idx_shard($self, $smsg->{num} % $self->{shards});
161         $idx->index_raw($msgref, $mime, $smsg);
162         my $n = $self->{transact_bytes} += $smsg->{raw_bytes};
163         $n >= ($PublicInbox::SearchIdx::BATCH_BYTES * $self->{shards});
164 }
165
166 sub _add {
167         my ($self, $mime, $check_cb) = @_;
168
169         # spam check:
170         if ($check_cb) {
171                 $mime = $check_cb->($mime, $self->{ibx}) or return;
172         }
173
174         # All pipes (> $^F) known to Perl 5.6+ have FD_CLOEXEC set,
175         # as does SQLite 3.4.1+ (released in 2007-07-20), and
176         # Xapian 1.3.2+ (released 2015-03-15).
177         # For the most part, we can spawn git-fast-import without
178         # leaking FDs to it...
179         $self->idx_init;
180
181         my ($num, $mid0) = v2_num_for($self, $mime);
182         defined $num or return; # duplicate
183         defined $mid0 or die "BUG: \$mid0 undefined\n";
184         my $im = $self->importer;
185         my $smsg = bless { mid => $mid0, num => $num }, 'PublicInbox::Smsg';
186         my $cmt = $im->add($mime, undef, $smsg); # sets $smsg->{ds|ts|blob}
187         $cmt = $im->get_mark($cmt);
188         $self->{last_commit}->[$self->{epoch_max}] = $cmt;
189
190         my $msgref = delete $smsg->{-raw_email};
191         if (do_idx($self, $msgref, $mime, $smsg)) {
192                 $self->checkpoint;
193         }
194
195         $cmt;
196 }
197
198 sub v2_num_for {
199         my ($self, $mime) = @_;
200         my $mids = mids($mime->header_obj);
201         if (@$mids) {
202                 my $mid = $mids->[0];
203                 my $num = $self->{mm}->mid_insert($mid);
204                 if (defined $num) { # common case
205                         return ($num, $mid);
206                 }
207
208                 # crap, Message-ID is already known, hope somebody just resent:
209                 foreach my $m (@$mids) {
210                         # read-only lookup now safe to do after above barrier
211                         # easy, don't store duplicates
212                         # note: do not add more diagnostic info here since
213                         # it gets noisy on public-inbox-watch restarts
214                         return () if content_exists($self, $mime, $m);
215                 }
216
217                 # AltId may pre-populate article numbers (e.g. X-Mail-Count
218                 # or NNTP article number), use that article number if it's
219                 # not in Over.
220                 my $altid = $self->{ibx}->{altid};
221                 if ($altid && grep(/:file=msgmap\.sqlite3\z/, @$altid)) {
222                         my $num = $self->{mm}->num_for($mid);
223
224                         if (defined $num && !$self->{over}->get_art($num)) {
225                                 return ($num, $mid);
226                         }
227                 }
228
229                 # very unlikely:
230                 warn "<$mid> reused for mismatched content\n";
231
232                 # try the rest of the mids
233                 for(my $i = $#$mids; $i >= 1; $i--) {
234                         my $m = $mids->[$i];
235                         $num = $self->{mm}->mid_insert($m);
236                         if (defined $num) {
237                                 warn "alternative <$m> for <$mid> found\n";
238                                 return ($num, $m);
239                         }
240                 }
241         }
242         # none of the existing Message-IDs are good, generate a new one:
243         v2_num_for_harder($self, $mime);
244 }
245
246 sub v2_num_for_harder {
247         my ($self, $mime) = @_;
248
249         my $hdr = $mime->header_obj;
250         my $dig = content_digest($mime);
251         my $mid0 = PublicInbox::Import::digest2mid($dig, $hdr);
252         my $num = $self->{mm}->mid_insert($mid0);
253         unless (defined $num) {
254                 # it's hard to spoof the last Received: header
255                 my @recvd = $hdr->header_raw('Received');
256                 $dig->add("Received: $_") foreach (@recvd);
257                 $mid0 = PublicInbox::Import::digest2mid($dig, $hdr);
258                 $num = $self->{mm}->mid_insert($mid0);
259
260                 # fall back to a random Message-ID and give up determinism:
261                 until (defined($num)) {
262                         $dig->add(rand);
263                         $mid0 = PublicInbox::Import::digest2mid($dig, $hdr);
264                         warn "using random Message-ID <$mid0> as fallback\n";
265                         $num = $self->{mm}->mid_insert($mid0);
266                 }
267         }
268         PublicInbox::Import::append_mid($hdr, $mid0);
269         ($num, $mid0);
270 }
271
272 sub idx_shard {
273         my ($self, $shard_i) = @_;
274         $self->{idx_shards}->[$shard_i];
275 }
276
277 sub _idx_init { # with_umask callback
278         my ($self, $opt) = @_;
279         $self->lock_acquire unless $opt && $opt->{-skip_lock};
280         $self->{over}->create;
281
282         # xcpdb can change shard count while -watch is idle
283         my $nshards = count_shards($self);
284         $self->{shards} = $nshards if $nshards && $nshards != $self->{shards};
285
286         # need to create all shards before initializing msgmap FD
287         # idx_shards must be visible to all forked processes
288         my $max = $self->{shards} - 1;
289         my $idx = $self->{idx_shards} = [];
290         push @$idx, PublicInbox::SearchIdxShard->new($self, $_) for (0..$max);
291
292         # Now that all subprocesses are up, we can open the FDs
293         # for SQLite:
294         my $mm = $self->{mm} = PublicInbox::Msgmap->new_file(
295                                 "$self->{ibx}->{inboxdir}/msgmap.sqlite3",
296                                 $self->{ibx}->{-no_sync} ? 2 : 1);
297         $mm->{dbh}->begin_work;
298 }
299
300 # idempotent
301 sub idx_init {
302         my ($self, $opt) = @_;
303         return if $self->{idx_shards};
304         my $ibx = $self->{ibx};
305
306         # do not leak read-only FDs to child processes, we only have these
307         # FDs for duplicate detection so they should not be
308         # frequently activated.
309         delete @$ibx{qw(mm search)};
310         $ibx->git->cleanup;
311
312         $self->{parallel} = 0 if ($ibx->{indexlevel}//'') eq 'basic';
313         if ($self->{parallel}) {
314                 pipe(my ($r, $w)) or die "pipe failed: $!";
315                 # pipe for barrier notifications doesn't need to be big,
316                 # 1031: F_SETPIPE_SZ
317                 fcntl($w, 1031, 4096) if $^O eq 'linux';
318                 $self->{bnote} = [ $r, $w ];
319                 $w->autoflush(1);
320         }
321
322         $ibx->umask_prepare;
323         $ibx->with_umask(\&_idx_init, $self, $opt);
324 }
325
326 # returns an array mapping [ epoch => latest_commit ]
327 # latest_commit may be undef if nothing was done to that epoch
328 # $replace_map = { $object_id => $strref, ... }
329 sub _replace_oids ($$$) {
330         my ($self, $mime, $replace_map) = @_;
331         $self->done;
332         my $pfx = "$self->{ibx}->{inboxdir}/git";
333         my $rewrites = []; # epoch => commit
334         my $max = $self->{epoch_max};
335
336         unless (defined($max)) {
337                 defined(my $latest = git_dir_latest($self, \$max)) or return;
338                 $self->{epoch_max} = $max;
339         }
340
341         foreach my $i (0..$max) {
342                 my $git_dir = "$pfx/$i.git";
343                 -d $git_dir or next;
344                 my $git = PublicInbox::Git->new($git_dir);
345                 my $im = $self->import_init($git, 0, 1);
346                 $rewrites->[$i] = $im->replace_oids($mime, $replace_map);
347                 $im->done;
348         }
349         $rewrites;
350 }
351
352 sub content_hashes ($) {
353         my ($mime) = @_;
354         my @chashes = ( content_hash($mime) );
355
356         # We still support Email::MIME, here, and
357         # Email::MIME->as_string doesn't always round-trip, so we may
358         # use a second content_hash
359         my $rt = content_hash(PublicInbox::Eml->new(\($mime->as_string)));
360         push @chashes, $rt if $chashes[0] ne $rt;
361         \@chashes;
362 }
363
364 sub content_matches ($$) {
365         my ($chashes, $existing) = @_;
366         my $chash = content_hash($existing);
367         foreach (@$chashes) {
368                 return 1 if $_ eq $chash
369         }
370         0
371 }
372
373 # used for removing or replacing (purging)
374 sub rewrite_internal ($$;$$$) {
375         my ($self, $old_eml, $cmt_msg, $new_eml, $sref) = @_;
376         $self->idx_init;
377         my ($im, $need_reindex, $replace_map);
378         if ($sref) {
379                 $replace_map = {}; # oid => sref
380                 $need_reindex = [] if $new_eml;
381         } else {
382                 $im = $self->importer;
383         }
384         my $over = $self->{over};
385         my $chashes = content_hashes($old_eml);
386         my $removed = [];
387         my $mids = mids($old_eml->header_obj);
388
389         # We avoid introducing new blobs into git since the raw content
390         # can be slightly different, so we do not need the user-supplied
391         # message now that we have the mids and content_hash
392         $old_eml = undef;
393         my $mark;
394
395         foreach my $mid (@$mids) {
396                 my %gone; # num => [ smsg, $mime, raw ]
397                 my ($id, $prev);
398                 while (my $smsg = $over->next_by_mid($mid, \$id, \$prev)) {
399                         my $msg = get_blob($self, $smsg);
400                         if (!defined($msg)) {
401                                 warn "broken smsg for $mid\n";
402                                 next; # continue
403                         }
404                         my $orig = $$msg;
405                         my $cur = PublicInbox::Eml->new($msg);
406                         if (content_matches($chashes, $cur)) {
407                                 $gone{$smsg->{num}} = [ $smsg, $cur, \$orig ];
408                         }
409                 }
410                 my $n = scalar keys %gone;
411                 next unless $n;
412                 if ($n > 1) {
413                         warn "BUG: multiple articles linked to <$mid>\n",
414                                 join(',', sort keys %gone), "\n";
415                 }
416                 foreach my $num (keys %gone) {
417                         my ($smsg, $mime, $orig) = @{$gone{$num}};
418                         # $removed should only be set once assuming
419                         # no bugs in our deduplication code:
420                         $removed = [ undef, $mime, $smsg ];
421                         my $oid = $smsg->{blob};
422                         if ($replace_map) {
423                                 $replace_map->{$oid} = $sref;
424                         } else {
425                                 ($mark, undef) = $im->remove($orig, $cmt_msg);
426                                 $removed->[0] = $mark;
427                         }
428                         $orig = undef;
429                         if ($need_reindex) { # ->replace
430                                 push @$need_reindex, $smsg;
431                         } else { # ->purge or ->remove
432                                 $self->{mm}->num_delete($num);
433                         }
434                         unindex_oid_remote($self, $oid, $mid);
435                 }
436         }
437
438         if (defined $mark) {
439                 my $cmt = $im->get_mark($mark);
440                 $self->{last_commit}->[$self->{epoch_max}] = $cmt;
441         }
442         if ($replace_map && scalar keys %$replace_map) {
443                 my $rewrites = _replace_oids($self, $new_eml, $replace_map);
444                 return { rewrites => $rewrites, need_reindex => $need_reindex };
445         }
446         defined($mark) ? $removed : undef;
447 }
448
449 # public (see PublicInbox::Import->remove), but note the 3rd element
450 # (retval[2]) is not part of the stable API shared with Import->remove
451 sub remove {
452         my ($self, $eml, $cmt_msg) = @_;
453         my $r = $self->{ibx}->with_umask(\&rewrite_internal,
454                                                 $self, $eml, $cmt_msg);
455         defined($r) && defined($r->[0]) ? @$r: undef;
456 }
457
458 sub _replace ($$;$$) {
459         my ($self, $old_eml, $new_eml, $sref) = @_;
460         my $arg = [ $self, $old_eml, undef, $new_eml, $sref ];
461         my $rewritten = $self->{ibx}->with_umask(\&rewrite_internal,
462                         $self, $old_eml, undef, $new_eml, $sref) or return;
463
464         my $rewrites = $rewritten->{rewrites};
465         # ->done is called if there are rewrites since we gc+prune from git
466         $self->idx_init if @$rewrites;
467
468         for my $i (0..$#$rewrites) {
469                 defined(my $cmt = $rewrites->[$i]) or next;
470                 $self->{last_commit}->[$i] = $cmt;
471         }
472         $rewritten;
473 }
474
475 # public
476 sub purge {
477         my ($self, $mime) = @_;
478         my $rewritten = _replace($self, $mime, undef, \'') or return;
479         $rewritten->{rewrites}
480 }
481
482 # returns the git object_id of $fh, does not write the object to FS
483 sub git_hash_raw ($$) {
484         my ($self, $raw) = @_;
485         # grab the expected OID we have to reindex:
486         pipe(my($in, $w)) or die "pipe: $!";
487         my $git_dir = $self->{ibx}->git->{git_dir};
488         my $cmd = ['git', "--git-dir=$git_dir", qw(hash-object --stdin)];
489         my $r = popen_rd($cmd, undef, { 0 => $in });
490         print $w $$raw or die "print \$w: $!";
491         close $w or die "close \$w: $!";
492         local $/ = "\n";
493         chomp(my $oid = <$r>);
494         close $r or die "git hash-object failed: $?";
495         $oid =~ /\A$OID\z/ or die "OID not expected: $oid";
496         $oid;
497 }
498
499 sub _check_mids_match ($$$) {
500         my ($old_list, $new_list, $hdrs) = @_;
501         my %old_mids = map { $_ => 1 } @$old_list;
502         my %new_mids = map { $_ => 1 } @$new_list;
503         my @old = keys %old_mids;
504         my @new = keys %new_mids;
505         my $err = "$hdrs may not be changed when replacing\n";
506         die $err if scalar(@old) != scalar(@new);
507         delete @new_mids{@old};
508         delete @old_mids{@new};
509         die $err if (scalar(keys %old_mids) || scalar(keys %new_mids));
510 }
511
512 # Changing Message-IDs or References with ->replace isn't supported.
513 # The rules for dealing with messages with multiple or conflicting
514 # Message-IDs are pretty complex and rethreading hasn't been fully
515 # implemented, yet.
516 sub check_mids_match ($$) {
517         my ($old_mime, $new_mime) = @_;
518         my $old = $old_mime->header_obj;
519         my $new = $new_mime->header_obj;
520         _check_mids_match(mids($old), mids($new), 'Message-ID(s)');
521         _check_mids_match(references($old), references($new),
522                         'References/In-Reply-To');
523 }
524
525 # public
526 sub replace ($$$) {
527         my ($self, $old_mime, $new_mime) = @_;
528
529         check_mids_match($old_mime, $new_mime);
530
531         # mutt will always add Content-Length:, Status:, Lines: when editing
532         PublicInbox::Import::drop_unwanted_headers($new_mime);
533
534         my $raw = $new_mime->as_string;
535         my $expect_oid = git_hash_raw($self, \$raw);
536         my $rewritten = _replace($self, $old_mime, $new_mime, \$raw) or return;
537         my $need_reindex = $rewritten->{need_reindex};
538
539         # just in case we have bugs in deduplication code:
540         my $n = scalar(@$need_reindex);
541         if ($n > 1) {
542                 my $list = join(', ', map {
543                                         "$_->{num}: <$_->{mid}>"
544                                 } @$need_reindex);
545                 warn <<"";
546 W: rewritten $n messages matching content of original message (expected: 1).
547 W: possible bug in public-inbox, NNTP article IDs and Message-IDs follow:
548 W: $list
549
550         }
551
552         # make sure we really got the OID:
553         my ($blob, $type, $bytes) = $self->{ibx}->git->check($expect_oid);
554         $blob eq $expect_oid or die "BUG: $expect_oid not found after replace";
555
556         # don't leak FDs to Xapian:
557         $self->{ibx}->git->cleanup;
558
559         # reindex modified messages:
560         for my $smsg (@$need_reindex) {
561                 my $new_smsg = bless {
562                         blob => $blob,
563                         raw_bytes => $bytes,
564                         num => $smsg->{num},
565                         mid => $smsg->{mid},
566                 }, 'PublicInbox::Smsg';
567                 my $sync = { autime => $smsg->{ds}, cotime => $smsg->{ts} };
568                 $new_smsg->populate($new_mime, $sync);
569                 do_idx($self, \$raw, $new_mime, $new_smsg);
570         }
571         $rewritten->{rewrites};
572 }
573
574 sub last_epoch_commit ($$;$) {
575         my ($self, $i, $cmt) = @_;
576         my $v = PublicInbox::Search::SCHEMA_VERSION();
577         $self->{mm}->last_commit_xap($v, $i, $cmt);
578 }
579
580 sub set_last_commits ($) {
581         my ($self) = @_;
582         defined(my $epoch_max = $self->{epoch_max}) or return;
583         my $last_commit = $self->{last_commit};
584         foreach my $i (0..$epoch_max) {
585                 defined(my $cmt = $last_commit->[$i]) or next;
586                 $last_commit->[$i] = undef;
587                 last_epoch_commit($self, $i, $cmt);
588         }
589 }
590
591 sub barrier_init {
592         my ($self, $n) = @_;
593         $self->{bnote} or return;
594         --$n;
595         my $barrier = { map { $_ => 1 } (0..$n) };
596 }
597
598 sub barrier_wait {
599         my ($self, $barrier) = @_;
600         my $bnote = $self->{bnote} or return;
601         my $r = $bnote->[0];
602         while (scalar keys %$barrier) {
603                 defined(my $l = readline($r)) or die "EOF on barrier_wait: $!";
604                 $l =~ /\Abarrier (\d+)/ or die "bad line on barrier_wait: $l";
605                 delete $barrier->{$1} or die "bad shard[$1] on barrier wait";
606         }
607 }
608
609 # public
610 sub checkpoint ($;$) {
611         my ($self, $wait) = @_;
612
613         if (my $im = $self->{im}) {
614                 if ($wait) {
615                         $im->barrier;
616                 } else {
617                         $im->checkpoint;
618                 }
619         }
620         my $shards = $self->{idx_shards};
621         if ($shards) {
622                 my $dbh = $self->{mm}->{dbh};
623
624                 # SQLite msgmap data is second in importance
625                 $dbh->commit;
626
627                 # SQLite overview is third
628                 $self->{over}->commit_lazy;
629
630                 # Now deal with Xapian
631                 if ($wait) {
632                         my $barrier = $self->barrier_init(scalar @$shards);
633
634                         # each shard needs to issue a barrier command
635                         $_->remote_barrier for @$shards;
636
637                         # wait for each Xapian shard
638                         $self->barrier_wait($barrier);
639                 } else {
640                         $_->remote_commit for @$shards;
641                 }
642
643                 # last_commit is special, don't commit these until
644                 # remote shards are done:
645                 $dbh->begin_work;
646                 set_last_commits($self);
647                 $dbh->commit;
648
649                 $dbh->begin_work;
650         }
651         $self->{total_bytes} += $self->{transact_bytes};
652         $self->{transact_bytes} = 0;
653 }
654
655 # issue a write barrier to ensure all data is visible to other processes
656 # and read-only ops.  Order of data importance is: git > SQLite > Xapian
657 # public
658 sub barrier { checkpoint($_[0], 1) };
659
660 # public
661 sub done {
662         my ($self) = @_;
663         my $err = '';
664         if (my $im = delete $self->{im}) {
665                 eval { $im->done }; # PublicInbox::Import::done
666                 $err .= "import done: $@\n" if $@;
667         }
668         if (!$err) {
669                 eval { checkpoint($self) };
670                 $err .= "checkpoint: $@\n" if $@;
671         }
672         if (my $mm = delete $self->{mm}) {
673                 my $m = $err ? 'rollback' : 'commit';
674                 eval { $mm->{dbh}->$m };
675                 $err .= "msgmap $m: $@\n" if $@;
676         }
677         my $shards = delete $self->{idx_shards};
678         if ($shards) {
679                 for (@$shards) {
680                         eval { $_->remote_close };
681                         $err .= "shard close: $@\n" if $@;
682                 }
683         }
684         eval { $self->{over}->disconnect };
685         $err .= "over disconnect: $@\n" if $@;
686         delete $self->{bnote};
687         my $nbytes = $self->{total_bytes};
688         $self->{total_bytes} = 0;
689         $self->lock_release(!!$nbytes) if $shards;
690         $self->{ibx}->git->cleanup;
691         die $err if $err;
692 }
693
694 sub fill_alternates ($$) {
695         my ($self, $epoch) = @_;
696
697         my $pfx = "$self->{ibx}->{inboxdir}/git";
698         my $all = "$self->{ibx}->{inboxdir}/all.git";
699         PublicInbox::Import::init_bare($all) unless -d $all;
700         my $info_dir = "$all/objects/info";
701         my $alt = "$info_dir/alternates";
702         my (%alt, $new);
703         my $mode = 0644;
704         if (-e $alt) {
705                 open(my $fh, '<', $alt) or die "open < $alt: $!\n";
706                 $mode = (stat($fh))[2] & 07777;
707
708                 # we assign a sort score to every alternate and favor
709                 # the newest (highest numbered) one because loose objects
710                 # require scanning epochs and only the latest epoch is
711                 # expected to see loose objects
712                 my $score;
713                 my $other = 0; # in case admin adds non-epoch repos
714                 %alt = map {;
715                         if (m!\A\Q../../\E([0-9]+)\.git/objects\z!) {
716                                 $score = $1 + 0;
717                         } else {
718                                 $score = --$other;
719                         }
720                         $_ => $score;
721                 } split(/\n+/, do { local $/; <$fh> });
722         }
723
724         foreach my $i (0..$epoch) {
725                 my $dir = "../../git/$i.git/objects";
726                 if (!exists($alt{$dir}) && -d "$pfx/$i.git") {
727                         $alt{$dir} = $i;
728                         $new = 1;
729                 }
730         }
731         return unless $new;
732
733         my ($fh, $tmp) = tempfile('alt-XXXXXXXX', DIR => $info_dir);
734         print $fh join("\n", sort { $alt{$b} <=> $alt{$a} } keys %alt), "\n"
735                 or die "print $tmp: $!\n";
736         chmod($mode, $fh) or die "fchmod $tmp: $!\n";
737         close $fh or die "close $tmp $!\n";
738         rename($tmp, $alt) or die "rename $tmp => $alt: $!\n";
739 }
740
741 sub git_init {
742         my ($self, $epoch) = @_;
743         my $git_dir = "$self->{ibx}->{inboxdir}/git/$epoch.git";
744         PublicInbox::Import::init_bare($git_dir);
745         my @cmd = (qw/git config/, "--file=$git_dir/config",
746                         'include.path', '../../all.git/config');
747         PublicInbox::Import::run_die(\@cmd);
748         fill_alternates($self, $epoch);
749         $git_dir
750 }
751
752 sub git_dir_latest {
753         my ($self, $max) = @_;
754         $$max = -1;
755         my $pfx = "$self->{ibx}->{inboxdir}/git";
756         return unless -d $pfx;
757         my $latest;
758         opendir my $dh, $pfx or die "opendir $pfx: $!\n";
759         while (defined(my $git_dir = readdir($dh))) {
760                 $git_dir =~ m!\A([0-9]+)\.git\z! or next;
761                 if ($1 > $$max) {
762                         $$max = $1;
763                         $latest = "$pfx/$git_dir";
764                 }
765         }
766         $latest;
767 }
768
769 sub importer {
770         my ($self) = @_;
771         my $im = $self->{im};
772         if ($im) {
773                 if ($im->{bytes_added} < $self->{rotate_bytes}) {
774                         return $im;
775                 } else {
776                         $self->{im} = undef;
777                         $im->done;
778                         $im = undef;
779                         $self->checkpoint;
780                         my $git_dir = $self->git_init(++$self->{epoch_max});
781                         my $git = PublicInbox::Git->new($git_dir);
782                         return $self->import_init($git, 0);
783                 }
784         }
785         my $epoch = 0;
786         my $max;
787         my $latest = git_dir_latest($self, \$max);
788         if (defined $latest) {
789                 my $git = PublicInbox::Git->new($latest);
790                 my $packed_bytes = $git->packed_bytes;
791                 my $unpacked_bytes = $packed_bytes / $PACKING_FACTOR;
792
793                 if ($unpacked_bytes >= $self->{rotate_bytes}) {
794                         $epoch = $max + 1;
795                 } else {
796                         $self->{epoch_max} = $max;
797                         return $self->import_init($git, $packed_bytes);
798                 }
799         }
800         $self->{epoch_max} = $epoch;
801         $latest = $self->git_init($epoch);
802         $self->import_init(PublicInbox::Git->new($latest), 0);
803 }
804
805 sub import_init {
806         my ($self, $git, $packed_bytes, $tmp) = @_;
807         my $im = PublicInbox::Import->new($git, undef, undef, $self->{ibx});
808         $im->{bytes_added} = int($packed_bytes / $PACKING_FACTOR);
809         $im->{lock_path} = undef;
810         $im->{path_type} = 'v2';
811         $self->{im} = $im unless $tmp;
812         $im;
813 }
814
815 # XXX experimental
816 sub diff ($$$) {
817         my ($mid, $cur, $new) = @_;
818
819         my ($ah, $an) = tempfile('email-cur-XXXXXXXX', TMPDIR => 1);
820         print $ah $cur->as_string or die "print: $!";
821         close $ah or die "close: $!";
822         my ($bh, $bn) = tempfile('email-new-XXXXXXXX', TMPDIR => 1);
823         PublicInbox::Import::drop_unwanted_headers($new);
824         print $bh $new->as_string or die "print: $!";
825         close $bh or die "close: $!";
826         my $cmd = [ qw(diff -u), $an, $bn ];
827         print STDERR "# MID conflict <$mid>\n";
828         my $pid = spawn($cmd, undef, { 1 => 2 });
829         waitpid($pid, 0) == $pid or die "diff did not finish";
830         unlink($an, $bn);
831 }
832
833 sub get_blob ($$) {
834         my ($self, $smsg) = @_;
835         if (my $im = $self->{im}) {
836                 my $msg = $im->cat_blob($smsg->{blob});
837                 return $msg if $msg;
838         }
839         # older message, should be in alternates
840         $self->{ibx}->msg_by_smsg($smsg);
841 }
842
843 sub content_exists ($$$) {
844         my ($self, $mime, $mid) = @_;
845         my $over = $self->{over};
846         my $chashes = content_hashes($mime);
847         my ($id, $prev);
848         while (my $smsg = $over->next_by_mid($mid, \$id, \$prev)) {
849                 my $msg = get_blob($self, $smsg);
850                 if (!defined($msg)) {
851                         warn "broken smsg for $mid\n";
852                         next;
853                 }
854                 my $cur = PublicInbox::Eml->new($msg);
855                 return 1 if content_matches($chashes, $cur);
856
857                 # XXX DEBUG_DIFF is experimental and may be removed
858                 diff($mid, $cur, $mime) if $ENV{DEBUG_DIFF};
859         }
860         undef;
861 }
862
863 sub atfork_child {
864         my ($self) = @_;
865         if (my $shards = $self->{idx_shards}) {
866                 $_->atfork_child foreach @$shards;
867         }
868         if (my $im = $self->{im}) {
869                 $im->atfork_child;
870         }
871         die "unexpected mm" if $self->{mm};
872         close $self->{bnote}->[0] or die "close bnote[0]: $!\n";
873         $self->{bnote}->[1];
874 }
875
876 sub reindex_checkpoint ($$) {
877         my ($self, $sync) = @_;
878
879         $self->{ibx}->git->cleanup; # *async_wait
880         ${$sync->{need_checkpoint}} = 0;
881         $sync->{mm_tmp}->atfork_prepare;
882         $self->done; # release lock
883
884         if (my $pr = $sync->{-opt}->{-progress}) {
885                 $pr->(sprintf($sync->{-regen_fmt}, ${$sync->{nr}}));
886         }
887
888         # allow -watch or -mda to write...
889         $self->idx_init; # reacquire lock
890         $sync->{mm_tmp}->atfork_parent;
891 }
892
893 sub index_oid { # cat_async callback
894         my ($bref, $oid, $type, $size, $arg) = @_;
895         return if $size == 0; # purged
896         my ($num, $mid0);
897         my $mime = PublicInbox::Eml->new($$bref);
898         my $mids = mids($mime->header_obj);
899         my $chash = content_hash($mime);
900         my $self = $arg->{v2w};
901
902         if (scalar(@$mids) == 0) {
903                 warn "E: $oid has no Message-ID, skipping\n";
904                 return;
905         }
906
907         # {unindexed} is unlikely
908         if ((my $unindexed = $arg->{unindexed}) && scalar(@$mids) == 1) {
909                 $num = delete($unindexed->{$mids->[0]});
910                 if (defined $num) {
911                         $mid0 = $mids->[0];
912                         $self->{mm}->mid_set($num, $mid0);
913                         delete($arg->{unindexed}) if !keys(%$unindexed);
914                 }
915         }
916         if (!defined($num)) { # reuse if reindexing (or duplicates)
917                 my $over = $self->{over};
918                 for my $mid (@$mids) {
919                         ($num, $mid0) = $over->num_mid0_for_oid($oid, $mid);
920                         last if defined $num;
921                 }
922         }
923         $mid0 //= do { # is this a number we got before?
924                 $num = $arg->{mm_tmp}->num_for($mids->[0]);
925                 defined($num) ? $mids->[0] : undef;
926         };
927         if (!defined($num)) {
928                 for (my $i = $#$mids; $i >= 1; $i--) {
929                         $num = $arg->{mm_tmp}->num_for($mids->[$i]);
930                         if (defined($num)) {
931                                 $mid0 = $mids->[$i];
932                                 last;
933                         }
934                 }
935         }
936         if (defined($num)) {
937                 $arg->{mm_tmp}->num_delete($num);
938         } else { # never seen
939                 $num = $self->{mm}->mid_insert($mids->[0]);
940                 if (defined($num)) {
941                         $mid0 = $mids->[0];
942                 } else { # rare, try the rest of them, backwards
943                         for (my $i = $#$mids; $i >= 1; $i--) {
944                                 $num = $self->{mm}->mid_insert($mids->[$i]);
945                                 if (defined($num)) {
946                                         $mid0 = $mids->[$i];
947                                         last;
948                                 }
949                         }
950                 }
951         }
952         if (!defined($num)) {
953                 warn "E: $oid <", join('> <', @$mids), "> is a duplicate\n";
954                 return;
955         }
956         ++${$arg->{nr}};
957         my $smsg = bless {
958                 raw_bytes => $size,
959                 num => $num,
960                 blob => $oid,
961                 mid => $mid0,
962         }, 'PublicInbox::Smsg';
963         $smsg->populate($mime, $arg);
964         if (do_idx($self, $bref, $mime, $smsg)) {
965                 ${$arg->{need_checkpoint}} = 1;
966         }
967 }
968
969 # only update last_commit for $i on reindex iff newer than current
970 sub update_last_commit ($$$$) {
971         my ($self, $git, $i, $cmt) = @_;
972         my $last = last_epoch_commit($self, $i);
973         if (defined $last && is_ancestor($git, $last, $cmt)) {
974                 my @cmd = (qw(rev-list --count), "$last..$cmt");
975                 chomp(my $n = $git->qx(@cmd));
976                 return if $n ne '' && $n == 0;
977         }
978         last_epoch_commit($self, $i, $cmt);
979 }
980
981 sub git_dir_n ($$) { "$_[0]->{ibx}->{inboxdir}/git/$_[1].git" }
982
983 sub last_commits ($$) {
984         my ($self, $epoch_max) = @_;
985         my $heads = [];
986         for (my $i = $epoch_max; $i >= 0; $i--) {
987                 $heads->[$i] = last_epoch_commit($self, $i);
988         }
989         $heads;
990 }
991
992 # returns a revision range for git-log(1)
993 sub log_range ($$$$$) {
994         my ($self, $sync, $git, $i, $tip) = @_;
995         my $opt = $sync->{-opt};
996         my $pr = $opt->{-progress} if (($opt->{verbose} || 0) > 1);
997         my $cur = $sync->{ranges}->[$i] or do {
998                 $pr->("$i.git indexing all of $tip") if $pr;
999                 return $tip; # all of it
1000         };
1001
1002         # fast equality check to avoid (v)fork+execve overhead
1003         if ($cur eq $tip) {
1004                 $sync->{ranges}->[$i] = undef;
1005                 return;
1006         }
1007
1008         my $range = "$cur..$tip";
1009         $pr->("$i.git checking contiguity... ") if $pr;
1010         if (is_ancestor($git, $cur, $tip)) { # common case
1011                 $pr->("OK\n") if $pr;
1012                 my $n = $git->qx(qw(rev-list --count), $range);
1013                 chomp($n);
1014                 if ($n == 0) {
1015                         $sync->{ranges}->[$i] = undef;
1016                         $pr->("$i.git has nothing new\n") if $pr;
1017                         return; # nothing to do
1018                 }
1019                 $pr->("$i.git has $n changes since $cur\n") if $pr;
1020         } else {
1021                 $pr->("FAIL\n") if $pr;
1022                 warn <<"";
1023 discontiguous range: $range
1024 Rewritten history? (in $git->{git_dir})
1025
1026                 chomp(my $base = $git->qx('merge-base', $tip, $cur));
1027                 if ($base) {
1028                         $range = "$base..$tip";
1029                         warn "found merge-base: $base\n"
1030                 } else {
1031                         $range = $tip;
1032                         warn "discarding history at $cur\n";
1033                 }
1034                 warn <<"";
1035 reindexing $git->{git_dir} starting at
1036 $range
1037
1038                 $sync->{unindex_range}->{$i} = "$base..$cur";
1039         }
1040         $range;
1041 }
1042
1043 sub sync_prepare ($$$) {
1044         my ($self, $sync, $epoch_max) = @_;
1045         my $pr = $sync->{-opt}->{-progress};
1046         my $regen_max = 0;
1047         my $head = $self->{ibx}->{ref_head} || 'refs/heads/master';
1048
1049         # reindex stops at the current heads and we later rerun index_sync
1050         # without {reindex}
1051         my $reindex_heads = last_commits($self, $epoch_max) if $sync->{reindex};
1052
1053         for (my $i = $epoch_max; $i >= 0; $i--) {
1054                 my $git_dir = git_dir_n($self, $i);
1055                 -d $git_dir or next; # missing epochs are fine
1056                 my $git = PublicInbox::Git->new($git_dir);
1057                 if ($reindex_heads) {
1058                         $head = $reindex_heads->[$i] or next;
1059                 }
1060                 chomp(my $tip = $git->qx(qw(rev-parse -q --verify), $head));
1061
1062                 next if $?; # new repo
1063                 my $range = log_range($self, $sync, $git, $i, $tip) or next;
1064                 # can't use 'rev-list --count' if we use --diff-filter
1065                 $pr->("$i.git counting $range ... ") if $pr;
1066                 # Don't bump num_highwater on --reindex by using {D}.
1067                 # We intentionally do NOT use {D} in the non-reindex case
1068                 # because we want NNTP article number gaps from unindexed
1069                 # messages to show up in mirrors, too.
1070                 $sync->{D} //= $sync->{reindex} ? {} : undef; # OID_BIN => NR
1071                 my $stk = log2stack($sync, $git, $range, $self->{ibx});
1072                 my $nr = $stk ? $stk->num_records : 0;
1073                 $pr->("$nr\n") if $pr;
1074                 $sync->{stacks}->[$i] = $stk if $stk;
1075                 $regen_max += $nr;
1076         }
1077
1078         # XXX this should not happen unless somebody bypasses checks in
1079         # our code and blindly injects "d" file history into git repos
1080         if (my @leftovers = keys %{delete($sync->{D}) // {}}) {
1081                 warn('W: unindexing '.scalar(@leftovers)." leftovers\n");
1082                 my $arg = { v2w => $self };
1083                 my $all = $self->{ibx}->git;
1084                 for my $oid (@leftovers) {
1085                         $oid = unpack('H*', $oid);
1086                         $self->{current_info} = "leftover $oid";
1087                         $all->cat_async($oid, \&unindex_oid, $arg);
1088                 }
1089                 $all->cat_async_wait;
1090         }
1091         return 0 if (!$regen_max && !keys(%{$self->{unindex_range}}));
1092
1093         # reindex should NOT see new commits anymore, if we do,
1094         # it's a problem and we need to notice it via die()
1095         my $pad = length($regen_max) + 1;
1096         $sync->{-regen_fmt} = "% ${pad}u/$regen_max\n";
1097         $sync->{nr} = \(my $nr = 0);
1098         return -1 if $sync->{reindex};
1099         $regen_max + $self->{mm}->num_highwater() || 0;
1100 }
1101
1102 sub unindex_oid_remote ($$$) {
1103         my ($self, $oid, $mid) = @_;
1104         my @removed = $self->{over}->remove_oid($oid, $mid);
1105         for my $num (@removed) {
1106                 my $idx = idx_shard($self, $num % $self->{shards});
1107                 $idx->remote_remove($oid, $num);
1108         }
1109 }
1110
1111 sub unindex_oid ($$;$) { # git->cat_async callback
1112         my ($bref, $oid, $type, $size, $sync) = @_;
1113         my $self = $sync->{v2w};
1114         my $unindexed = $sync->{in_unindex} ? $sync->{unindexed} : undef;
1115         my $mm = $self->{mm};
1116         my $mids = mids(PublicInbox::Eml->new($bref)->header_obj);
1117         undef $$bref;
1118         my $over = $self->{over};
1119         foreach my $mid (@$mids) {
1120                 my %gone;
1121                 my ($id, $prev);
1122                 while (my $smsg = $over->next_by_mid($mid, \$id, \$prev)) {
1123                         $gone{$smsg->{num}} = 1 if $oid eq $smsg->{blob};
1124                 }
1125                 my $n = scalar(keys(%gone)) or next;
1126                 if ($n > 1) {
1127                         warn "BUG: multiple articles linked to $oid\n",
1128                                 join(',',sort keys %gone), "\n";
1129                 }
1130                 foreach my $num (keys %gone) {
1131                         if ($unindexed) {
1132                                 my $mid0 = $mm->mid_for($num);
1133                                 $unindexed->{$mid0} = $num;
1134                         }
1135                         $mm->num_delete($num);
1136                 }
1137                 unindex_oid_remote($self, $oid, $mid);
1138         }
1139 }
1140
1141 # this is rare, it only happens when we get discontiguous history in
1142 # a mirror because the source used -purge or -edit
1143 sub unindex ($$$$) {
1144         my ($self, $sync, $git, $unindex_range) = @_;
1145         my $unindexed = $sync->{unindexed} //= {}; # $mid0 => $num
1146         my $before = scalar keys %$unindexed;
1147         # order does not matter, here:
1148         my @cmd = qw(log --raw -r
1149                         --no-notes --no-color --no-abbrev --no-renames);
1150         my $fh = $git->popen(@cmd, $unindex_range);
1151         my $all = $self->{ibx}->git;
1152         local $sync->{in_unindex} = 1;
1153         while (<$fh>) {
1154                 /\A:\d{6} 100644 $OID ($OID) [AM]\tm$/o or next;
1155                 $all->cat_async($1, \&unindex_oid, $sync);
1156         }
1157         close $fh or die "git log failed: \$?=$?";
1158         $all->cat_async_wait;
1159
1160         return unless $sync->{-opt}->{prune};
1161         my $after = scalar keys %$unindexed;
1162         return if $before == $after;
1163
1164         # ensure any blob can not longer be accessed via dumb HTTP
1165         PublicInbox::Import::run_die(['git', "--git-dir=$git->{git_dir}",
1166                 qw(-c gc.reflogExpire=now gc --prune=all --quiet)]);
1167 }
1168
1169 sub sync_ranges ($$$) {
1170         my ($self, $sync, $epoch_max) = @_;
1171         my $reindex = $sync->{reindex};
1172
1173         return last_commits($self, $epoch_max) unless $reindex;
1174         return [] if ref($reindex) ne 'HASH';
1175
1176         my $ranges = $reindex->{from}; # arrayref;
1177         if (ref($ranges) ne 'ARRAY') {
1178                 die 'BUG: $reindex->{from} not an ARRAY';
1179         }
1180         $ranges;
1181 }
1182
1183 sub index_epoch ($$$) {
1184         my ($self, $sync, $i) = @_;
1185
1186         my $git_dir = git_dir_n($self, $i);
1187         -d $git_dir or return; # missing epochs are fine
1188         my $git = PublicInbox::Git->new($git_dir);
1189         if (my $unindex_range = delete $sync->{unindex_range}->{$i}) { # rare
1190                 unindex($self, $sync, $git, $unindex_range);
1191         }
1192         defined(my $stk = $sync->{stacks}->[$i]) or return;
1193         $sync->{stacks}->[$i] = undef;
1194         my $all = $self->{ibx}->git;
1195         while (my ($f, $at, $ct, $oid) = $stk->pop_rec) {
1196                 $self->{current_info} = "$i.git $oid";
1197                 if ($f eq 'm') {
1198                         my $arg = { %$sync, autime => $at, cotime => $ct };
1199                         if ($sync->{index_max_size}) {
1200                                 $all->check_async($oid, \&check_size, $arg);
1201                         } else {
1202                                 $all->cat_async($oid, \&index_oid, $arg);
1203                         }
1204                 } elsif ($f eq 'd') {
1205                         $all->cat_async($oid, \&unindex_oid, $sync);
1206                 }
1207                 if (${$sync->{need_checkpoint}}) {
1208                         reindex_checkpoint($self, $sync);
1209                 }
1210         }
1211         $all->check_async_wait;
1212         $all->cat_async_wait;
1213         update_last_commit($self, $git, $i, $stk->{latest_cmt});
1214 }
1215
1216 # public, called by public-inbox-index
1217 sub index_sync {
1218         my ($self, $opt) = @_;
1219         $opt ||= {};
1220         my $pr = $opt->{-progress};
1221         my $epoch_max;
1222         my $latest = git_dir_latest($self, \$epoch_max);
1223         return unless defined $latest;
1224         $self->idx_init($opt); # acquire lock
1225         fill_alternates($self, $epoch_max);
1226         $self->{over}->rethread_prepare($opt);
1227         my $sync = {
1228                 need_checkpoint => \(my $bool = 0),
1229                 unindex_range => {}, # EPOCH => oid_old..oid_new
1230                 reindex => $opt->{reindex},
1231                 -opt => $opt,
1232                 v2w => $self,
1233         };
1234         $sync->{ranges} = sync_ranges($self, $sync, $epoch_max);
1235         if (sync_prepare($self, $sync, $epoch_max)) {
1236                 # tmp_clone seems to fail if inside a transaction, so
1237                 # we rollback here (because we opened {mm} for reading)
1238                 # Note: we do NOT rely on DBI transactions for atomicity;
1239                 # only for batch performance.
1240                 $self->{mm}->{dbh}->rollback;
1241                 $self->{mm}->{dbh}->begin_work;
1242                 $sync->{mm_tmp} =
1243                         $self->{mm}->tmp_clone($self->{ibx}->{inboxdir});
1244         }
1245         if ($sync->{index_max_size} = $self->{ibx}->{index_max_size}) {
1246                 $sync->{index_oid} = \&index_oid;
1247         }
1248         # work forwards through history
1249         index_epoch($self, $sync, $_) for (0..$epoch_max);
1250         $self->done;
1251
1252         if (my $nr = $sync->{nr}) {
1253                 my $pr = $sync->{-opt}->{-progress};
1254                 $pr->('all.git '.sprintf($sync->{-regen_fmt}, $$nr)) if $pr;
1255         }
1256         $self->{over}->rethread_done($opt);
1257
1258         # reindex does not pick up new changes, so we rerun w/o it:
1259         if ($opt->{reindex}) {
1260                 my %again = %$opt;
1261                 $sync = undef;
1262                 delete @again{qw(rethread reindex -skip_lock)};
1263                 index_sync($self, \%again);
1264         }
1265 }
1266
1267 1;