Page MenuHomePhabricator

Fix loop in QueryIterator when row count is an exact multiple of page size
ClosedPublic

Authored by epriestley on Jun 7 2018, 7:14 PM.
Tags
None
Referenced Files
F15459154: D19479.id.diff
Mon, Mar 31, 10:16 AM
F15443230: D19479.id46586.diff
Thu, Mar 27, 3:10 AM
F15442758: D19479.id46590.diff
Thu, Mar 27, 12:17 AM
F15442757: D19479.id46586.diff
Thu, Mar 27, 12:17 AM
F15442756: D19479.id.diff
Thu, Mar 27, 12:17 AM
F15438687: D19479.id46590.diff
Wed, Mar 26, 2:43 AM
F15438175: D19479.diff
Tue, Mar 25, 11:40 PM
F15432904: D19479.id.diff
Mon, Mar 24, 8:10 PM
Subscribers
None

Details

Summary

Ref T13152. The pager does a bit of magic here and doesn't populate nextPageID when it knows it got an exact final page. The logic misfired in this case and sent us back to the start.

Test Plan
  • Set page size to 1 to guarantee rows were an exact multiple of page size.
  • Ran rebuild-identities (I no-op'd the actual logic to make it faster).
  • Before: looped forever.
  • After: clean exit after processing everything.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable