]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/LeiImport.pm
lei: always open mail_sync.sqlite3 R/W
[public-inbox.git] / lib / PublicInbox / LeiImport.pm
index f9a46ec58b5233b877b5ab01c084bbcd354c2a23..b9865829e39d6e7f5c7131ee772f99d13e74928a 100644 (file)
@@ -1,4 +1,4 @@
-# Copyright (C) 2021 all contributors <meta@public-inbox.org>
+# Copyright (C) all contributors <meta@public-inbox.org>
 # License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
 
 # front-end for the "lei import" sub-command
@@ -6,6 +6,7 @@ package PublicInbox::LeiImport;
 use strict;
 use v5.10.1;
 use parent qw(PublicInbox::IPC PublicInbox::LeiInput);
+use PublicInbox::InboxWritable qw(eml_from_path);
 
 # /^input_/ subs are used by (or override) PublicInbox::LeiInput superclass
 
@@ -15,7 +16,7 @@ sub input_eml_cb { # used by PublicInbox::LeiInput::input_fh
        if (my $all_vmd = $self->{all_vmd}) {
                @$vmd{keys %$all_vmd} = values %$all_vmd;
        }
-       $self->{lei}->{sto}->ipc_do('set_eml', $eml, $vmd, $xoids);
+       $self->{lei}->{sto}->wq_do('set_eml', $eml, $vmd, $xoids);
 }
 
 sub input_mbox_cb { # MboxReader callback
@@ -28,32 +29,40 @@ sub input_mbox_cb { # MboxReader callback
        input_eml_cb($self, $eml, $vmd);
 }
 
-sub input_maildir_cb { # maildir_each_eml cb
-       my ($f, $kw, $eml, $self) = @_;
+sub pmdir_cb { # called via wq_io_do from LeiPmdir->each_mdir_fn
+       my ($self, $f, $fl) = @_;
+       my ($folder, $bn) = ($f =~ m!\A(.+?)/(?:new|cur)/([^/]+)\z!) or
+               die "BUG: $f was not from a Maildir?\n";
+       my $kw = PublicInbox::MdirReader::flags2kw($fl);
+       substr($folder, 0, 0) = 'maildir:'; # add prefix
+       my $lse = $self->{lse} //= $self->{lei}->{sto}->search;
+       my $lms = $self->{-lms_rw} //= $self->{lei}->lms; # may be 0 or undef
+       my @oidbin = $lms ? $lms->name_oidbin($folder, $bn) : ();
+       @oidbin > 1 and warn("W: $folder/*/$$bn not unique:\n",
+                               map { "\t".unpack('H*', $_)."\n" } @oidbin);
+       my %seen;
+       my @docids = sort { $a <=> $b } grep { !$seen{$_}++ }
+                       map { $lse->over->oidbin_exists($_) } @oidbin;
        my $vmd = $self->{-import_kw} ? { kw => $kw } : undef;
-       if ($self->{-mail_sync}) {
-               if ($f =~ m!\A(.+?)/(?:new|cur)/([^/]+)\z!) { # ugh...
-                       $vmd->{sync_info} = [ "maildir:$1", \(my $n = $2) ];
-               } else {
-                       warn "E: $f was not from a Maildir?\n";
-               }
+       if (scalar @docids) {
+               $lse->kw_changed(undef, $kw, \@docids) or return;
+       }
+       if (my $eml = eml_from_path($f)) {
+               $vmd->{sync_info} = [ $folder, \$bn ] if $self->{-mail_sync};
+               $self->input_eml_cb($eml, $vmd);
        }
-       $self->input_eml_cb($eml, $vmd);
 }
 
 sub input_net_cb { # imap_each / nntp_each
        my ($uri, $uid, $kw, $eml, $self) = @_;
-       my $vmd = $self->{-import_kw} ? { kw => $kw } : undef;
-       $vmd->{sync_info} = [ $$uri, $uid ] if $self->{-mail_sync};
        if (defined $eml) {
+               my $vmd = $self->{-import_kw} ? { kw => $kw } : undef;
+               $vmd->{sync_info} = [ $$uri, $uid ] if $self->{-mail_sync};
                $self->input_eml_cb($eml, $vmd);
-       } elsif ($vmd) { # old message, kw only
-               my $oid = $self->{-lms_ro}->imap_oid2($uri, $uid) // return;
-               my @docids = $self->{lse}->over->blob_exists($oid) or return;
-               my $lei = $self->{lei};
-               $lei->qerr("# $oid => @$kw\n") if $lei->{opt}->{verbose};
-               $self->{lei}->{sto}->ipc_do('set_eml_vmd', undef,
-                                               $vmd, \@docids);
+       } elsif (my $ikw = $self->{lei}->{ikw}) { # old message, kw only
+               # we send $uri as a bare SCALAR and not a URIimap ref to
+               # reduce socket traffic:
+               $ikw->wq_io_do('ck_update_kw', [], $$uri, $uid, $kw);
        }
 }
 
@@ -65,35 +74,37 @@ sub do_import_index ($$@) {
        my $vmd_mod = $self->vmd_mod_extract(\@inputs);
        return $lei->fail(join("\n", @{$vmd_mod->{err}})) if $vmd_mod->{err};
        $self->{all_vmd} = $vmd_mod if scalar keys %$vmd_mod;
-       $self->prepare_inputs($lei, \@inputs) or return;
+       $lei->ale; # initialize for workers to read (before LeiPmdir->new)
        $self->{-mail_sync} = $lei->{opt}->{'mail-sync'} // 1;
+       $self->prepare_inputs($lei, \@inputs) or return;
 
-       $lei->ale; # initialize for workers to read
-       my $j = $lei->{opt}->{jobs} // scalar(@{$self->{inputs}}) || 1;
-       if (my $net = $lei->{net}) {
+       my $j = $lei->{opt}->{jobs} // 0;
+       $j =~ /\A([0-9]+),[0-9]+\z/ and $j = $1 + 0;
+       $j ||= scalar(@{$self->{inputs}}) || 1;
+       my $ikw;
+       my $net = $lei->{net};
+       if ($net) {
                # $j = $net->net_concurrency($j); TODO
                if ($lei->{opt}->{incremental} // 1) {
                        $net->{incremental} = 1;
-                       $net->{-lms_ro} = $sto->search->lms // 0;
-                       if ($self->{-import_kw}) {
+                       $net->{-lms_rw} = $lei->lms // 0;
+                       if ($self->{-import_kw} && $net->{-lms_rw} &&
+                                       !$lei->{opt}->{'new-only'} &&
+                                       $net->{imap_order}) {
+                               require PublicInbox::LeiImportKw;
+                               $ikw = PublicInbox::LeiImportKw->new($lei);
                                $net->{each_old} = 1;
-                               $self->{-lms_ro} = $net->{-lms_ro};
-                               $self->{lse} = $sto->search;
                        }
                }
        } else {
                my $nproc = $self->detect_nproc;
                $j = $nproc if $j > $nproc;
        }
-       my $ops = {};
-       $lei->{auth}->op_merge($ops, $self) if $lei->{auth};
-       $self->{-wq_nr_workers} = $j // 1; # locked
+       ($lei->{opt}->{'new-only'} && (!$net || !$net->{imap_order})) and
+               warn "# --new-only is only for IMAP\n";
        $lei->{-eml_noisy} = 1;
-       (my $op_c, $ops) = $lei->workers_start($self, $j, $ops);
-       $lei->{wq1} = $self;
        $lei->{-err_type} = 'non-fatal';
-       net_merge_all_done($self) unless $lei->{auth};
-       $op_c->op_wait_event($ops);
+       $lei->wq1_start($self, $j);
 }
 
 sub lei_import { # the main "lei import" method
@@ -104,16 +115,20 @@ sub lei_import { # the main "lei import" method
 
 sub _complete_import {
        my ($lei, @argv) = @_;
-       my $sto = $lei->_lei_store or return;
-       my $lms = $sto->search->lms or return;
-       my $match_cb = $lei->complete_url_prepare(\@argv);
-       map { $match_cb->($_) } $lms->folders;
+       my ($re, $cur, $match_cb) = $lei->complete_url_prepare(\@argv);
+       my @k = $lei->url_folder_cache->keys($argv[-1] // undef, 1);
+       my @m = map { $match_cb->($_) } @k;
+       my %f = map { $_ => 1 } (@m ? @m : @k);
+       if (my $lms = $lei->lms) {
+               @k = $lms->folders($argv[-1] // undef, 1);
+               @m = map { $match_cb->($_) } @k;
+               if (@m) { @f{@m} = @m } else { @f{@k} = @k }
+       }
+       keys %f;
 }
 
 no warnings 'once';
 *ipc_atfork_child = \&PublicInbox::LeiInput::input_only_atfork_child;
 *net_merge_all_done = \&PublicInbox::LeiInput::input_only_net_merge_all_done;
 
-# the following works even when LeiAuth is lazy-loaded
-*net_merge_all = \&PublicInbox::LeiAuth::net_merge_all;
 1;