From 942d6d60d5d7a7e8ea28447e3007770f341fbd30 Mon Sep 17 00:00:00 2001
From: epriestley <git@epriestley.com>
Date: Wed, 30 Jan 2019 19:49:02 -0800
Subject: [PATCH] Don't load unnecessary handle data on "transaction.search"

Summary: Ref T13242. Currently, the transaction query loads handles by default (this is unusual). We don't need them, so turn them off.

Test Plan: No apparent behavioral change, will compare production profiles.

Reviewers: amckinley

Reviewed By: amckinley

Maniphest Tasks: T13242

Differential Revision: https://secure.phabricator.com/D20068
---
 .../transactions/conduit/TransactionSearchConduitAPIMethod.php   | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/applications/transactions/conduit/TransactionSearchConduitAPIMethod.php b/src/applications/transactions/conduit/TransactionSearchConduitAPIMethod.php
index 0394432ff3..0edc0b3f5a 100644
--- a/src/applications/transactions/conduit/TransactionSearchConduitAPIMethod.php
+++ b/src/applications/transactions/conduit/TransactionSearchConduitAPIMethod.php
@@ -68,6 +68,7 @@ final class TransactionSearchConduitAPIMethod
       $object);
 
     $xaction_query
+      ->needHandles(false)
       ->withObjectPHIDs(array($object->getPHID()))
       ->setViewer($viewer);