From 77704e711685d119fde9d1737029e102c76c9a14 Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Tue, 24 Apr 2018 19:43:25 +0000 Subject: [PATCH] msgmap: add limit to response for NNTP All callers in expect to iterate through results. This was causing unfairness when fetching large ranges via XHDR as rtin does :< Fixes: b8c41362f2a5c8fc "nntp: simplify the long_response API" --- lib/PublicInbox/Msgmap.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/PublicInbox/Msgmap.pm b/lib/PublicInbox/Msgmap.pm index 192e311a..fdc71e46 100644 --- a/lib/PublicInbox/Msgmap.pm +++ b/lib/PublicInbox/Msgmap.pm @@ -199,7 +199,7 @@ sub msg_range { my $attr = { Columns => [] }; my $mids = $dbh->selectall_arrayref(<<'', $attr, $$beg, $end); SELECT num,mid FROM msgmap WHERE num >= ? AND num <= ? -ORDER BY num ASC +ORDER BY num ASC LIMIT 1000 $$beg = $mids->[-1]->[0] + 1 if @$mids; $mids -- 2.44.0