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
Unknown Object (File)
Sun, Dec 15, 2:38 AM
Unknown Object (File)
Wed, Dec 11, 5:43 PM
Unknown Object (File)
Wed, Dec 11, 9:04 AM
Unknown Object (File)
Tue, Dec 3, 11:47 PM
Unknown Object (File)
Tue, Dec 3, 11:46 PM
Unknown Object (File)
Tue, Dec 3, 11:46 PM
Unknown Object (File)
Tue, Dec 3, 11:22 PM
Unknown Object (File)
Tue, Dec 3, 2:42 PM
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
Branch
attach5
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 12845
Build 16369: Run Core Tests
Build 16368: arc lint + arc unit

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.