From 7b5471b1aea249a0d78b1e7480d0b5e2a9831b5a Mon Sep 17 00:00:00 2001 From: Wez Furlong Date: Wed, 28 Aug 2013 13:29:36 -0700 Subject: [PATCH] fix perf regression when using 'owned' query Summary: D6335 has some unexpected side effects. This adds back the where clause for the owned query. There may be other problems. Test Plan: Ran: ``` echo '{"query":"owned","guids":["myphid"]}' | arc --conduit-uri=https://myhost call-conduit differential.find ``` Reviewers: epriestley, dschleimer Reviewed By: epriestley CC: Korvin, aran Differential Revision: https://secure.phabricator.com/D6832 --- .../conduit/ConduitAPI_differential_find_Method.php | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/applications/differential/conduit/ConduitAPI_differential_find_Method.php b/src/applications/differential/conduit/ConduitAPI_differential_find_Method.php index 52ec54a87a..c74a37b6bd 100644 --- a/src/applications/differential/conduit/ConduitAPI_differential_find_Method.php +++ b/src/applications/differential/conduit/ConduitAPI_differential_find_Method.php @@ -69,6 +69,9 @@ final class ConduitAPI_differential_find_Method extends ConduitAPIMethod { $query ->withIDs($guids); break; + case 'owned': + $query->withAuthors($guids); + break; case 'phids': $query ->withPHIDs($guids);