diff options
author | Eric Wong <e@80x24.org> | 2020-09-02 11:04:16 +0000 |
---|---|---|
committer | Eric Wong <e@80x24.org> | 2020-09-03 20:11:03 +0000 |
commit | 80b887f29b2ec71d025b4c266a1c26314758994c (patch) | |
tree | 31f9937daf20c21b33c0407763bf2d6bd06d2f66 /lib/PublicInbox/Mbox.pm | |
parent | 851aa7183cf2b152d7ff187729bc896597d22fad (diff) | |
download | public-inbox-80b887f29b2ec71d025b4c266a1c26314758994c.tar.gz |
search: replace ->query with ->mset
Nearly all of the search uses in the production code rely on a Xapian mset iterator being returned (instead of an array of $smsg objects). So default to returning the mset and move the burden of smsg array conversion into the test cases.
Diffstat (limited to 'lib/PublicInbox/Mbox.pm')
-rw-r--r-- | lib/PublicInbox/Mbox.pm | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/PublicInbox/Mbox.pm b/lib/PublicInbox/Mbox.pm index 0223bead..47025891 100644 --- a/lib/PublicInbox/Mbox.pm +++ b/lib/PublicInbox/Mbox.pm @@ -213,7 +213,7 @@ sub results_cb { } # refill result set my $srch = $ctx->{-inbox}->search(undef, $ctx) or return; - my $mset = $srch->query($ctx->{query}, $ctx->{qopts}); + my $mset = $srch->mset($ctx->{query}, $ctx->{qopts}); my $size = $mset->size or return; $ctx->{qopts}->{offset} += $size; $ctx->{ids} = $srch->mset_to_artnums($mset); @@ -235,7 +235,7 @@ sub results_thread_cb { # refill result set my $srch = $ctx->{-inbox}->search(undef, $ctx) or return; - my $mset = $srch->query($ctx->{query}, $ctx->{qopts}); + my $mset = $srch->mset($ctx->{query}, $ctx->{qopts}); my $size = $mset->size or return; $ctx->{qopts}->{offset} += $size; $ctx->{ids} = $srch->mset_to_artnums($mset); @@ -254,7 +254,7 @@ sub mbox_all { my $qopts = $ctx->{qopts} = { mset => 2 }; # order by docid $qopts->{thread} = 1 if $q->{t}; - my $mset = $srch->query($q_string, $qopts); + my $mset = $srch->mset($q_string, $qopts); $qopts->{offset} = $mset->size or return [404, [qw(Content-Type text/plain)], ["No results found\n"]]; |