Page MenuHomePhabricator

Refine available filters and defaults for relationship selection
ClosedPublic

Authored by epriestley on Jun 30 2016, 5:56 PM.
Tags
None
Referenced Files
F14058142: D16202.diff
Sun, Nov 17, 9:40 AM
F13999677: D16202.diff
Thu, Oct 24, 4:24 PM
F13974138: D16202.id.diff
Oct 18 2024, 4:15 AM
Unknown Object (File)
Oct 12 2024, 7:30 PM
Unknown Object (File)
Oct 7 2024, 6:49 PM
Unknown Object (File)
Sep 2 2024, 2:23 AM
Unknown Object (File)
Aug 27 2024, 4:15 AM
Unknown Object (File)
Aug 26 2024, 1:42 AM
Subscribers
None

Details

Summary

Ref T4788. Fixes T10703.

In the longer term I want to put this on top of ApplicationSearch, but that's somewhat complex and we're at a fairly good point to pause this feature for feedback.

Inch toward that instead: provide more appropriate filters and defaults without rebuilding the underlying engine. Specifically:

  • No "assigned" for commits (barely makes sense).
  • No "assigned" for mocks (does not make sense).
  • Default to "open" for parent tasks, subtasks, close as duplicate, and merge into.

Also, add a key to the search_document table to improve the performance of the "all open stuff of type X" query. "All Open Tasks" is about 100x faster on my machine with this key.

Test Plan
  • Clicked all object relationships, saw more sensible filters and defaults.
  • Saw "open" query about 100x faster locally (300ms to 3ms).

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Refine available filters and defualts for relationship selection.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley retitled this revision from Refine available filters and defualts for relationship selection to Refine available filters and defaults for relationship selection.Jun 30 2016, 5:57 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Jun 30 2016, 6:48 PM
This revision was automatically updated to reflect the committed changes.