]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/LeiXSearch.pm
lei q: delay worker spawn
[public-inbox.git] / lib / PublicInbox / LeiXSearch.pm
index 95862306cd7a98beba3aca94659f17bd7bf3460a..ab66717c04e7b8142b70b386235011468aeb7780 100644 (file)
@@ -107,6 +107,18 @@ sub wait_startq ($) {
        read($startq, my $query_prepare_done, 1);
 }
 
+sub mset_progress {
+       my $lei = shift;
+       return unless $lei->{-progress};
+       if ($lei->{pkt_op}) { # called via pkt_op/pkt_do from workers
+               pkt_do($lei->{pkt_op}, 'mset_progress', @_);
+       } else { # single lei-daemon consumer
+               my ($desc, $mset_size, $mset_total_est) = @_;
+               $lei->{-mset_total} += $mset_size;
+               $lei->err("# $desc $mset_size/$mset_total_est");
+       }
+}
+
 sub query_thread_mset { # for --thread
        my ($self, $lei, $ibxish) = @_;
        local $0 = "$0 query_thread_mset";
@@ -121,7 +133,7 @@ sub query_thread_mset { # for --thread
        my $each_smsg = $lei->{ovv}->ovv_each_smsg_cb($lei, $ibxish);
        do {
                $mset = $srch->mset($mo->{qstr}, $mo);
-               pkt_do($lei->{pkt_op}, 'mset_progress', $desc, $mset->size,
+               mset_progress($lei, $desc, $mset->size,
                                $mset->get_matches_estimated);
                my $ids = $srch->mset_to_artnums($mset, $mo);
                my $ctx = { ids => $ids };
@@ -154,7 +166,7 @@ sub query_mset { # non-parallel for non-"--thread" users
        my $each_smsg = $lei->{ovv}->ovv_each_smsg_cb($lei, $self);
        do {
                $mset = $self->mset($mo->{qstr}, $mo);
-               pkt_do($lei->{pkt_op}, 'mset_progress', 'xsearch',
+               mset_progress($lei, 'xsearch', $mset->size,
                                $mset->size, $mset->get_matches_estimated);
                for my $mitem ($mset->items) {
                        my $smsg = smsg_for($self, $mitem) or next;
@@ -174,8 +186,8 @@ sub each_eml { # callback for MboxReader->mboxrd
        $smsg->{$_} //= '' for qw(from to cc ds subject references mid);
        delete @$smsg{qw(From Subject -ds -ts)};
        if (my $startq = delete($lei->{startq})) { wait_startq($startq) }
-       ++$lei->{-nr_remote_eml};
-       if (!$lei->{opt}->{quiet}) {
+       if ($lei->{-progress}) {
+               ++$lei->{-nr_remote_eml};
                my $now = now();
                my $next = $lei->{-next_progress} //= ($now + 1);
                if ($now > $next) {
@@ -198,6 +210,7 @@ sub query_remote_mboxrd {
        my ($self, $lei, $uris) = @_;
        local $0 = "$0 query_remote_mboxrd";
        $lei->atfork_child_wq($self);
+       local $SIG{TERM} = sub { exit(0) }; # for DESTROY (File::Temp, $reap)
        my ($opt, $env) = @$lei{qw(opt env)};
        my @qform = (q => $lei->{mset_opt}->{qstr}, x => 'm');
        push(@qform, t => 1) if $opt->{thread};
@@ -250,24 +263,21 @@ sub query_remote_mboxrd {
                shift(@$cmd) if !$cmd->[0];
 
                $lei->err("# @$cmd") if $verbose;
-               $? = 0;
-               my $fh = popen_rd($cmd, $env, $rdr);
+               my ($fh, $pid) = popen_rd($cmd, $env, $rdr);
                $fh = IO::Uncompress::Gunzip->new($fh);
-               eval {
-                       PublicInbox::MboxReader->mboxrd($fh, \&each_eml, $self,
-                                                       $lei, $each_smsg);
-               };
-               return $lei->fail("E: @$cmd: $@") if $@;
+               PublicInbox::MboxReader->mboxrd($fh, \&each_eml, $self,
+                                               $lei, $each_smsg);
+               waitpid($pid, 0) == $pid or die "BUG: waitpid (curl): $!";
                if ($? == 0) {
                        my $nr = $lei->{-nr_remote_eml};
-                       pkt_do($lei->{pkt_op}, 'mset_progress',
-                               $lei->{-current_url}, $nr, $nr);
+                       mset_progress($lei, $lei->{-current_url}, $nr, $nr);
                        next;
                }
                seek($cerr, $coff, SEEK_SET) or warn "seek(curl stderr): $!\n";
                my $e = do { local $/; <$cerr> } //
                                die "read(curl stderr): $!\n";
                $coff += length($e);
+               truncate($cerr, 0);
                next if (($? >> 8) == 22 && $e =~ /\b404\b/);
                $lei->child_error($?);
                $uri->query_form(q => $lei->{mset_opt}->{qstr});
@@ -316,19 +326,11 @@ Error closing $lei->{ovv}->{dst}: $!
                }
                $lei->start_mua;
        }
-       $lei->{opt}->{quiet} or
+       $lei->{-progress} and
                $lei->err('# ', $lei->{-mset_total} // 0, " matches");
        $lei->dclose;
 }
 
-sub mset_progress { # called via pkt_op/pkt_do from workers
-       my ($lei, $pargs) = @_;
-       my ($desc, $mset_size, $mset_total_est) = @$pargs;
-       return if $lei->{opt}->{quiet};
-       $lei->{-mset_total} += $mset_size;
-       $lei->err("# $desc $mset_size/$mset_total_est");
-}
-
 sub do_post_augment {
        my ($lei, $zpipe, $au_done) = @_;
        my $l2m = $lei->{l2m} or die 'BUG: no {l2m}';
@@ -344,7 +346,6 @@ sub do_post_augment {
 }
 
 my $MAX_PER_HOST = 4;
-sub MAX_PER_HOST { $MAX_PER_HOST }
 
 sub concurrency {
        my ($self, $opt) = @_;
@@ -386,8 +387,9 @@ sub query_prepare { # called by wq_do
 
 sub fail_handler ($;$$) {
        my ($lei, $code, $io) = @_;
-       if (my $lxs = delete $lei->{lxs}) {
-               $lxs->wq_wait_old($lei) if $lxs->wq_kill_old; # lei-daemon
+       for my $f (qw(lxs l2m)) {
+               my $wq = delete $lei->{$f} or next;
+               $wq->wq_wait_old($lei) if $wq->wq_kill_old; # lei-daemon
        }
        close($io) if $io; # needed to avoid warnings on SIGPIPE
        $lei->x_it($code // (1 >> 8));
@@ -400,23 +402,32 @@ sub sigpipe_handler { # handles SIGPIPE from l2m/lxs workers
 sub do_query {
        my ($self, $lei) = @_;
        $lei->{1}->autoflush(1);
+       $lei->start_pager if -t $lei->{1};
+       $lei->{ovv}->ovv_begin($lei);
        my ($au_done, $zpipe);
        my $l2m = $lei->{l2m};
+       $lei->atfork_prepare_wq($self);
+       $self->wq_workers_start('lei_xsearch', $self->{jobs}, $lei->oldset);
+       delete $self->{-ipc_atfork_child_close};
        if ($l2m) {
+               $lei->atfork_prepare_wq($l2m);
+               $l2m->wq_workers_start('lei2mail', $l2m->{jobs}, $lei->oldset);
+               delete $l2m->{-ipc_atfork_child_close};
                pipe($lei->{startq}, $au_done) or die "pipe: $!";
                # 1031: F_SETPIPE_SZ
                fcntl($lei->{startq}, 1031, 4096) if $^O eq 'linux';
                $zpipe = $l2m->pre_augment($lei);
        }
-       my $in_loop = exists $lei->{sock};
        my $ops = {
                '|' => [ \&sigpipe_handler, $lei ],
                '!' => [ \&fail_handler, $lei ],
                '.' => [ \&do_post_augment, $lei, $zpipe, $au_done ],
                '' => [ \&query_done, $lei ],
                'mset_progress' => [ \&mset_progress, $lei ],
+               'x_it' => [ $lei->can('x_it'), $lei ],
+               'child_error' => [ $lei->can('child_error'), $lei ],
        };
-       (my $op, $lei->{pkt_op}) = PublicInbox::PktOp->pair($ops, $in_loop);
+       (my $op, $lei->{pkt_op}) = PublicInbox::PktOp->pair($ops);
        my ($lei_ipc, @io) = $lei->atfork_parent_wq($self);
        delete($lei->{pkt_op});
 
@@ -427,7 +438,7 @@ sub do_query {
        }
        start_query($self, \@io, $lei_ipc);
        $self->wq_close(1);
-       unless ($in_loop) {
+       if ($lei->{oneshot}) {
                # for the $lei_ipc->atfork_child_wq PIPE handler:
                while ($op->{sock}) { $op->event_step }
        }