Page MenuHomePhabricator

Fix pagination of fulltext search results
ClosedPublic

Authored by epriestley on Apr 12 2017, 9:58 PM.
Tags
None
Referenced Files
F15459867: D17667.id.diff
Mon, Mar 31, 5:31 PM
F15449774: D17667.id42499.diff
Fri, Mar 28, 12:12 PM
F15447447: D17667.id.diff
Thu, Mar 27, 11:55 PM
F15446518: D17667.id42490.diff
Thu, Mar 27, 7:24 PM
F15436571: D17667.diff
Tue, Mar 25, 3:15 PM
F15433598: D17667.id.diff
Mon, Mar 24, 11:59 PM
F15407305: D17667.diff
Tue, Mar 18, 4:49 PM
F15381989: D17667.id.diff
Fri, Mar 14, 10:14 AM
Subscribers
None

Details

Summary

Fixes T8285. Fulltext search relies on an underlying engine which can not realistically use cursor paging. This is unusual and creates some oddness.

Tweak a few numbers -- and how offsets are handled -- to separate the filtered offset and unfiltered offset.

Test Plan
  • Set page size to 2.
  • Ran a query.
  • Paged forward and backward through results sensibly, seeing the full result set.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable