Page MenuHomePhabricator

Modernize DifferentialRevisionSearchEngine
ClosedPublic

Authored by epriestley on May 15 2016, 5:37 PM.
Tags
None
Referenced Files
F18829702: D15921.id38337.diff
Sat, Oct 25, 12:07 AM
F18816789: D15921.id.diff
Tue, Oct 21, 9:25 AM
F18810192: D15921.diff
Sun, Oct 19, 8:03 PM
F18803700: D15921.id38337.diff
Sat, Oct 18, 2:57 AM
F18654174: D15921.id38348.diff
Sep 22 2025, 3:39 AM
F18629797: D15921.id.diff
Sep 16 2025, 9:55 AM
F18616288: D15921.id38337.diff
Sep 14 2025, 4:57 PM
F18604167: D15921.id38337.diff
Sep 13 2025, 6:13 PM
Subscribers
None

Details

Summary

Ref T10939. Ref T4144. This moves the revision SearchEngine to modern code so I can add some kind of bucketing layer on top of it.

This seems to have worked pretty cleanly. One thing is that I removed the ability to search for "pending drafts":

  • This was added in D1927 from a bootcamp task, was an indirect solution to a questionable problem, and almost certainly would not meet the bar today.
  • Later, in D3324, we added the icons to the list. I think this is a better solution in general. In particular, it specifically addressed the query being kind of junky.
  • At the time, Differential had a prebuilt "Drafts" filter. This was removed in D6347 with the move to ApplicationSearch, which simplified the large number of prebuilt filters. Although we got a lot of feedback about that, none requested that the drafts filter be restored.
Test Plan

Searched for responsible users, subscribers, orders, projects, repositories.

Diff Detail

Repository
rP Phabricator
Branch
dash1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 12211
Build 15418: Run Core Tests
Build 15417: arc lint + arc unit