]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/LeiStore.pm
lei: propagate keyword changes from lei/store
[public-inbox.git] / lib / PublicInbox / LeiStore.pm
index 3f33d114a6c7e85d5a3513696f2c86e00fffdfd1..a91b30f749520b9ad2eab309d00e168d79fba863 100644 (file)
@@ -25,9 +25,14 @@ use PublicInbox::MID qw(mids);
 use PublicInbox::LeiSearch;
 use PublicInbox::MDA;
 use PublicInbox::Spawn qw(spawn);
+use PublicInbox::MdirReader;
+use PublicInbox::LeiToMail;
 use List::Util qw(max);
 use File::Temp ();
 use POSIX ();
+use IO::Handle (); # ->autoflush
+use Sys::Syslog qw(syslog openlog);
+use Errno qw(EEXIST ENOENT);
 
 sub new {
        my (undef, $dir, $opt) = @_;
@@ -164,12 +169,92 @@ sub _docids_for ($$) {
        sort { $a <=> $b } values %docids;
 }
 
+# n.b. similar to LeiExportKw->export_kw_md, but this is for a single eml
+sub export1_kw_md ($$$$$) {
+       my ($self, $mdir, $bn, $oidbin, $vmdish) = @_; # vmd/vmd_mod
+       my $orig = $bn;
+       my (@try, $unkn, $kw);
+       if ($bn =~ s/:2,([a-zA-Z]*)\z//) {
+               ($kw, $unkn) = PublicInbox::MdirReader::flags2kw($1);
+               if (my $set = $vmdish->{kw}) {
+                       $kw = $set;
+               } elsif (my $add = $vmdish->{'+kw'}) {
+                       @$kw{@$add} = ();
+               } elsif (my $del = $vmdish->{-kw}) {
+                       delete @$kw{@$del};
+               } # else no changes...
+               @try = qw(cur new);
+       } else { # no keywords, yet, could be in new/
+               @try = qw(new cur);
+               $unkn = [];
+               if (my $set = $vmdish->{kw}) {
+                       $kw = $set;
+               } elsif (my $add = $vmdish->{'+kw'}) {
+                       @$kw{@$add} = (); # auto-vivify
+               } else { # ignore $vmdish->{-kw}
+                       $kw = [];
+               }
+       }
+       $kw = [ keys %$kw ] if ref($kw) eq 'HASH';
+       $bn .= ':2,'. PublicInbox::LeiToMail::kw2suffix($kw, @$unkn);
+       return if $orig eq $bn; # no change
+
+       # we use link(2) + unlink(2) since rename(2) may
+       # inadvertently clobber if the "uniquefilename" part wasn't
+       # actually unique.
+       my $dst = "$mdir/cur/$bn";
+       for my $d (@try) {
+               my $src = "$mdir/$d/$orig";
+               if (link($src, $dst)) {
+                       if (!unlink($src) and $! != ENOENT) {
+                               syslog('warning', "unlink($src): $!");
+                       }
+                       # TODO: verify oidbin?
+                       lms_mv_src($self, "maildir:$mdir",
+                                       $oidbin, \$orig, $bn);
+                       return;
+               } elsif ($! == EEXIST) { # lost race with "lei export-kw"?
+                       return;
+               } elsif ($! == ENOENT) {
+                       syslog('warning', "link($src -> $dst): $!")
+               } # else loop @try
+       }
+       my $e = $!;
+       my $src = "$mdir/{".join(',', @try)."}/$orig";
+       my $oidhex = unpack('H*', $oidbin);
+       syslog('warning', "link($src -> $dst) ($oidhex): $e");
+       for (@try) { return if -e "$mdir/$_/$orig" };
+       lms_clear_src($self, "maildir:$mdir", \$orig);
+}
+
+sub sto_export_kw ($$$) {
+       my ($self, $docid, $vmdish) = @_; # vmdish (vmd or vmd_mod)
+       my ($eidx, $tl) = eidx_init($self);
+       my $lms = _lms_rw($self) // return;
+       my $xr3 = $eidx->{oidx}->get_xref3($docid, 1);
+       for my $row (@$xr3) {
+               my (undef, undef, $oidbin) = @$row;
+               my $locs = $lms->locations_for($oidbin) // next;
+               while (my ($loc, $ids) = each %$locs) {
+                       if ($loc =~ s!\Amaildir:!!i) {
+                               for my $id (@$ids) {
+                                       export1_kw_md($self, $loc, $id,
+                                                       $oidbin, $vmdish);
+                               }
+                       }
+                       # TODO: IMAP
+               }
+       }
+}
+
+# vmd = { kw => [ qw(seen ...) ], L => [ qw(inbox ...) ] }
 sub set_eml_vmd {
        my ($self, $eml, $vmd, $docids) = @_;
        my ($eidx, $tl) = eidx_init($self);
        $docids //= [ _docids_for($self, $eml) ];
        for my $docid (@$docids) {
                $eidx->idx_shard($docid)->ipc_do('set_vmd', $docid, $vmd);
+               sto_export_kw($self, $docid, $vmd);
        }
        $docids;
 }
@@ -201,7 +286,7 @@ sub _lms_rw ($) {
                require PublicInbox::LeiMailSync;
                my $f = "$self->{priv_eidx}->{topdir}/mail_sync.sqlite3";
                my $lms = PublicInbox::LeiMailSync->new($f);
-               $lms->lms_begin;
+               $lms->lms_write_prepare;
                $lms;
        };
 }
@@ -216,9 +301,20 @@ sub lms_mv_src {
        _lms_rw($self)->mv_src($folder, $oidbin, $id, $newbn);
 }
 
+sub lms_forget_folders {
+       my ($self, @folders) = @_;
+       my $lms = _lms_rw($self);
+       for my $f (@folders) { $lms->forget_folder($f) }
+}
+
+sub lms_rename_folder {
+       my ($self, $old, $new) = @_;
+       _lms_rw($self)->rename_folder($old, $new);
+}
+
 sub set_sync_info {
        my ($self, $oidhex, $folder, $id) = @_;
-       _lms_rw($self)->set_src($oidhex, $folder, $id);
+       _lms_rw($self)->set_src(pack('H*', $oidhex), $folder, $id);
 }
 
 sub _remove_if_local { # git->cat_async arg
@@ -231,8 +327,8 @@ sub remove_docids ($;@) {
        my $eidx = eidx_init($self);
        for my $docid (@docids) {
                $eidx->idx_shard($docid)->ipc_do('xdb_remove', $docid);
-               $self->{oidx}->delete_by_num($docid);
-               $self->{oidx}->{dbh}->do(<<EOF, undef, $docid);
+               $eidx->{oidx}->delete_by_num($docid);
+               $eidx->{oidx}->{dbh}->do(<<EOF, undef, $docid);
 DELETE FROM xref3 WHERE docid = ?
 EOF
        }
@@ -272,6 +368,12 @@ EOF
        $docid;
 }
 
+sub _add_vmd ($$$$) {
+       my ($self, $idx, $docid, $vmd) = @_;
+       $idx->ipc_do('add_vmd', $docid, $vmd);
+       sto_export_kw($self, $docid, $vmd);
+}
+
 sub add_eml {
        my ($self, $eml, $vmd, $xoids) = @_;
        my $im = $self->{-fake_im} // $self->importer; # may create new epoch
@@ -298,7 +400,7 @@ sub add_eml {
                        @$vivify_xvmd = sort { $a <=> $b } keys(%docids);
                }
        }
-       if (@$vivify_xvmd) {
+       if (@$vivify_xvmd) { # docids list
                $xoids //= {};
                $xoids->{$smsg->{blob}} = 1;
                for my $docid (@$vivify_xvmd) {
@@ -315,7 +417,7 @@ sub add_eml {
                        for my $oid (keys %$xoids) {
                                $oidx->add_xref3($docid, -1, $oid, '.');
                        }
-                       $idx->ipc_do('add_vmd', $docid, $vmd) if $vmd;
+                       _add_vmd($self, $idx, $docid, $vmd) if $vmd;
                }
                $vivify_xvmd;
        } elsif (my @docids = _docids_for($self, $eml)) {
@@ -325,7 +427,7 @@ sub add_eml {
                        $oidx->add_xref3($docid, -1, $smsg->{blob}, '.');
                        # add_eidx_info for List-Id
                        $idx->ipc_do('add_eidx_info', $docid, '.', $eml);
-                       $idx->ipc_do('add_vmd', $docid, $vmd) if $vmd;
+                       _add_vmd($self, $idx, $docid, $vmd) if $vmd;
                }
                \@docids;
        } else { # totally new message
@@ -335,7 +437,7 @@ sub add_eml {
                $oidx->add_xref3($smsg->{num}, -1, $smsg->{blob}, '.');
                my $idx = $eidx->idx_shard($smsg->{num});
                $idx->index_eml($eml, $smsg);
-               $idx->ipc_do('add_vmd', $smsg->{num}, $vmd) if $vmd;
+               _add_vmd($self, $idx, $smsg->{num}, $vmd) if $vmd;
                $smsg;
        }
 }
@@ -353,6 +455,7 @@ sub index_eml_only {
        set_eml($self, $eml, $vmd, $xoids);
 }
 
+# store {kw} / {L} info for a message which is only in an external
 sub _external_only ($$$) {
        my ($self, $xoids, $eml) = @_;
        my $eidx = $self->{priv_eidx};
@@ -386,6 +489,7 @@ sub update_xvmd {
                next if $seen{$docid}++;
                my $idx = $eidx->idx_shard($docid);
                $idx->ipc_do('update_vmd', $docid, $vmd_mod);
+               sto_export_kw($self, $docid, $vmd_mod);
        }
        return unless scalar(keys(%$xoids));
 
@@ -398,12 +502,14 @@ sub update_xvmd {
                        }
                        my $idx = $eidx->idx_shard($docid);
                        $idx->ipc_do('update_vmd', $docid, $vmd_mod);
+                       sto_export_kw($self, $docid, $vmd_mod);
                }
                return;
        }
        # totally unseen
        my ($smsg, $idx) = _external_only($self, $xoids, $eml);
        $idx->ipc_do('update_vmd', $smsg->{num}, $vmd_mod);
+       sto_export_kw($self, $smsg->{num}, $vmd_mod);
 }
 
 # set or update keywords for external message, called via ipc_do
@@ -421,6 +527,7 @@ sub set_xvmd {
                next if $seen{$docid}++;
                my $idx = $eidx->idx_shard($docid);
                $idx->ipc_do('set_vmd', $docid, $vmd);
+               sto_export_kw($self, $docid, $vmd);
        }
        return unless scalar(keys(%$xoids));
 
@@ -431,6 +538,7 @@ sub set_xvmd {
        # totally unseen:
        my ($smsg, $idx) = _external_only($self, $xoids, $eml);
        $idx->ipc_do('add_vmd', $smsg->{num}, $vmd);
+       sto_export_kw($self, $smsg->{num}, $vmd);
 }
 
 sub checkpoint {
@@ -438,9 +546,7 @@ sub checkpoint {
        if (my $im = $self->{im}) {
                $wait ? $im->barrier : $im->checkpoint;
        }
-       if (my $lms = delete $self->{lms}) {
-               $lms->lms_commit;
-       }
+       delete $self->{lms};
        $self->{priv_eidx}->checkpoint($wait);
 }
 
@@ -460,7 +566,7 @@ sub xchg_stderr {
 }
 
 sub done {
-       my ($self) = @_;
+       my ($self, $sock_ref) = @_;
        my $err = '';
        if (my $im = delete($self->{im})) {
                eval { $im->done };
@@ -469,12 +575,14 @@ sub done {
                        warn $err;
                }
        }
-       if (my $lms = delete $self->{lms}) {
-               $lms->lms_commit;
-       }
+       delete $self->{lms};
        $self->{priv_eidx}->done; # V2Writable::done
        xchg_stderr($self);
        die $err if $err;
+
+       # notify clients ->done has been issued
+       defined($sock_ref) and
+               $self->{s2d_op_p}->pkt_do('sto_done_complete', $sock_ref);
 }
 
 sub ipc_atfork_child {
@@ -482,28 +590,39 @@ sub ipc_atfork_child {
        my $lei = $self->{lei};
        $lei->_lei_atfork_child(1) if $lei;
        xchg_stderr($self);
-       if (my $err = delete($self->{err_pipe})) {
-               close $err->[0];
-               $self->{-err_wr} = $err->[1];
+       if (my $to_close = delete($self->{to_close})) {
+               close($_) for @$to_close;
        }
+       openlog('lei/store', 'pid,nowait,nofatal,ndelay', 'user');
        $self->SUPER::ipc_atfork_child;
 }
 
 sub write_prepare {
        my ($self, $lei) = @_;
+       $lei // die 'BUG: $lei not passed';
        unless ($self->{-ipc_req}) {
-               my $d = $lei->store_path;
-               $self->ipc_lock_init("$d/ipc.lock");
-               substr($d, -length('/lei/store'), 10, '');
+               # s2d => store-to-daemon messages
+               require PublicInbox::PktOp;
+               my ($s2d_op_c, $s2d_op_p) = PublicInbox::PktOp->pair;
+               my $dir = $lei->store_path;
+               $self->ipc_lock_init("$dir/ipc.lock");
+               substr($dir, -length('/lei/store'), 10, '');
                pipe(my ($r, $w)) or die "pipe: $!";
-               my $err_pipe = [ $r, $w ];
+               $w->autoflush(1);
                # Mail we import into lei are private, so headers filtered out
                # by -mda for public mail are not appropriate
                local @PublicInbox::MDA::BAD_HEADERS = ();
-               $self->ipc_worker_spawn("lei/store $d", $lei->oldset,
-                                       { lei => $lei, err_pipe => $err_pipe });
+               $self->ipc_worker_spawn("lei/store $dir", $lei->oldset, {
+                                       lei => $lei,
+                                       -err_wr => $w,
+                                       to_close => [ $r, $s2d_op_c->{sock} ],
+                                       s2d_op_p => $s2d_op_p,
+                               });
                require PublicInbox::LeiStoreErr;
-               PublicInbox::LeiStoreErr->new($err_pipe->[0], $lei);
+               PublicInbox::LeiStoreErr->new($r, $lei);
+               $s2d_op_c->{ops} = {
+                       sto_done_complete => [ $lei->can('sto_done_complete') ]
+               };
        }
        $lei->{sto} = $self;
 }
@@ -515,4 +634,11 @@ sub git_blob_id { # called via LEI->git_blob_id
        git_sha(1, $eml)->hexdigest;
 }
 
+# called by lei-daemon before lei->refresh_watches
+sub add_sync_folders {
+       my ($self, @folders) = @_;
+       my $lms = _lms_rw($self);
+       for my $f (@folders) { $lms->fid_for($f, 1) }
+}
+
 1;