Page MenuHomePhabricator
Differential D12363

Modernize Phrequent and Commit query ordering/paging
ClosedPublic
Actions

Authored by epriestley on Apr 12 2015, 3:59 AM.
Tags
None
Referenced Files
F18818716: D12363.id29686.diff
Wed, Oct 22, 1:55 AM
F18809713: D12363.id29686.diff
Sun, Oct 19, 4:19 PM
F18765168: D12363.id29764.diff
Tue, Oct 7, 9:58 AM
F18627385: D12363.diff
Sep 16 2025, 3:52 AM
F18578434: D12363.id.diff
Sep 10 2025, 10:05 PM
F18566353: D12363.diff
Sep 9 2025, 2:31 PM
F18505360: D12363.id.diff
Sep 5 2025, 12:36 AM
F18497134: D12363.diff
Sep 4 2025, 5:53 PM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
Maniphest Tasks
Restricted Maniphest Task
T7803: Implement a composite query which unifies cursor paging across subqueries
Commits
Restricted Diffusion Commit
rP9c7c13ffc88a: Modernize Phrequent and Commit query ordering/paging
Summary

Ref T7803. Fixes T3870. Move these away from pagingColumn / reversePaging.

Test Plan
  • Tested/paged audit query.
  • Poked at Phrequent. Didn't seem any more broken than before.

Diff Detail

Repository
rP Phabricator
Branch
rpaging13
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 5246
Build 5264: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 29686.Apr 12 2015, 3:59 AM
epriestley retitled this revision from to Modernize Phrequent and Commit query ordering/paging.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added tasks: T7803: Implement a composite query which unifies cursor paging across subqueries, Restricted Maniphest Task.
Herald added a subscriber: epriestley. · View Herald TranscriptApr 12 2015, 4:00 AM
btrahan accepted this revision.Apr 13 2015, 5:31 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Apr 13 2015, 5:31 PM
Closed by commit rP9c7c13ffc88a: Modernize Phrequent and Commit query ordering/paging (authored by epriestley, committed by epriestley). · Explain WhyApr 13 2015, 7:00 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
diffusion/
query/
4 lines
phrequent/
query/
124 lines
infrastructure/
query/
policy/
22 lines

Diff 29686

View Options

src/applications/diffusion/query/DiffusionCommitQuery.php

Loading...
View Options

src/applications/phrequent/query/PhrequentUserTimeQuery.php

Loading...
View Options

src/infrastructure/query/policy/PhabricatorCursorPagedPolicyAwareQuery.php

Loading...
Phabricator · Built by Phacility