From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: X-Spam-Status: No, score=-4.0 required=3.0 tests=ALL_TRUSTED,AWL,BAYES_00 shortcircuit=no autolearn=ham autolearn_force=no version=3.4.0 Received: from localhost (dcvr.yhbt.net [127.0.0.1]) by dcvr.yhbt.net (Postfix) with ESMTP id 4803520179 for ; Fri, 17 Jun 2016 01:12:26 +0000 (UTC) From: Eric Wong To: meta@public-inbox.org Subject: [PATCH] search: increase limit for thread search Date: Fri, 17 Jun 2016 01:12:26 +0000 Message-Id: <20160617011226.8256-1-e@80x24.org> List-Id: Some threads are easily over 100 messages, so the 50 limit is not enough. It is likely that 1000 messages is not enough, either, and we will need to tune our threading to handle more messages and supply options for configurability. --- lib/PublicInbox/Search.pm | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lib/PublicInbox/Search.pm b/lib/PublicInbox/Search.pm index 0f7815f..d9fbc36 100644 --- a/lib/PublicInbox/Search.pm +++ b/lib/PublicInbox/Search.pm @@ -109,6 +109,8 @@ sub get_thread { my $path = id_compress($smsg->path); my $qsub = Search::Xapian::Query->new(xpfx('path').$path); my $query = Search::Xapian::Query->new(OP_OR, $qtid, $qsub); + $opts ||= {}; + $opts->{limit} ||= 1000; $self->do_enquire($query, $opts); }