X-Git-Url: http://www.git.stargrave.org/?a=blobdiff_plain;f=lib%2FPublicInbox%2FLeiQuery.pm;h=007e35fc41cf061b4ce963afc0ba99e0fda37d94;hb=7d2e572aca7297ea2015d2b6e7c71b672521ec82;hp=f71beae605aade29db986c514689f9684af0b73e;hpb=b503d87a23c90fc0f705a79bce34944e7e20b28b;p=public-inbox.git diff --git a/lib/PublicInbox/LeiQuery.pm b/lib/PublicInbox/LeiQuery.pm index f71beae6..007e35fc 100644 --- a/lib/PublicInbox/LeiQuery.pm +++ b/lib/PublicInbox/LeiQuery.pm @@ -5,20 +5,30 @@ package PublicInbox::LeiQuery; use strict; use v5.10.1; +use POSIX (); sub prep_ext { # externals_each callback my ($lxs, $exclude, $loc) = @_; $lxs->prepare_external($loc) unless $exclude->{$loc}; } -sub qstr_add { # for --stdin +sub _start_query { + my ($self) = @_; + if ($self->{net}) { + require PublicInbox::LeiAuth; + $self->{auth} = PublicInbox::LeiAuth->new + } + $self->{lxs}->do_query($self); +} + +sub qstr_add { # PublicInbox::InputPipe::consume callback for --stdin my ($self) = @_; # $_[1] = $rbuf if (defined($_[1])) { $_[1] eq '' and return eval { my $lse = delete $self->{lse}; $lse->query_approxidate($lse->git, $self->{mset_opt}->{qstr}); - $self->{lxs}->do_query($self); + _start_query($self); }; $self->{mset_opt}->{qstr} .= $_[1]; } else { @@ -41,8 +51,16 @@ sub lei_q { # we'll allow "--only $LOCATION --local" my $sto = $self->_lei_store(1); my $lse = $sto->search; + if (($opt->{'import-remote'} //= 1) | + (($opt->{'import-before'} //= \1) ? 1 : 0)) { + $sto->write_prepare($self); + } if ($opt->{'local'} //= scalar(@only) ? 0 : 1) { $lxs->prepare_external($lse); + } else { + my $tmp = PublicInbox::LeiXSearch->new; + $tmp->prepare_external($lse); + $self->ale->refresh_externals($tmp); } if (@only) { for my $loc (@only) { @@ -50,9 +68,12 @@ sub lei_q { $lxs->prepare_external($_) for @loc; } } else { + my (@ilocals, @iremotes); for my $loc (@{$opt->{include} // []}) { my @loc = $self->get_externals($loc) or return; $lxs->prepare_external($_) for @loc; + @ilocals = @{$lxs->{locals} // []}; + @iremotes = @{$lxs->{remotes} // []}; } # --external is enabled by default, but allow --no-external if ($opt->{external} //= 1) { @@ -64,28 +85,32 @@ sub lei_q { my $ne = $self->externals_each(\&prep_ext, $lxs, \%x); $opt->{remote} //= !($lxs->locals - $opt->{'local'}); if ($opt->{'local'}) { - delete($lxs->{remotes}) if !$opt->{remote}; + $lxs->{remotes} = \@iremotes if !$opt->{remote}; } else { - delete($lxs->{locals}); + $lxs->{locals} = \@ilocals; } } } unless ($lxs->locals || $lxs->remotes) { return $self->fail('no local or remote inboxes to search'); } + $self->ale->refresh_externals($lxs); my ($xj, $mj) = split(/,/, $opt->{jobs} // ''); if (defined($xj) && $xj ne '' && $xj !~ /\A[1-9][0-9]*\z/) { return $self->fail("`$xj' search jobs must be >= 1"); } $xj ||= $lxs->concurrency($opt); # allow: "--jobs ,$WRITER_ONLY" - my $nproc = $lxs->detect_nproc; # don't memoize, schedtool(1) exists + my $nproc = $lxs->detect_nproc // 1; # don't memoize, schedtool(1) exists $xj = $nproc if $xj > $nproc; - $lxs->{jobs} = $xj; + $lxs->{-wq_nr_workers} = $xj; if (defined($mj) && $mj !~ /\A[1-9][0-9]*\z/) { return $self->fail("`$mj' writer jobs must be >= 1"); } - $self->{l2m}->{jobs} = ($mj // $nproc) if $self->{l2m}; PublicInbox::LeiOverview->new($self) or return; + $self->{l2m} and $self->{l2m}->{-wq_nr_workers} = $mj // do { + $mj = POSIX::lround($nproc * 3 / 4); # keep some CPU for git + $mj <= 0 ? 1 : $mj; + }; my %mset_opt = map { $_ => $opt->{$_} } qw(threads limit offset); $mset_opt{asc} = $opt->{'reverse'} ? 1 : 0; @@ -115,7 +140,7 @@ no query allowed on command-line with --stdin return; } $mset_opt{qstr} = $lse->query_argv_to_string($lse->git, \@argv); - $lxs->do_query($self); + _start_query($self); } # shell completion helper called by lei__complete