Page MenuHomePhabricator
Create Task
Maniphest T344232

Query for global Feed Transaction Logs: Exception: ConpherenceTransactionQuery overheated
Closed, ResolvedPublic
Actions

Description

Upstreamed as https://we.phorge.it/T15642

  1. Go to https://phabricator.wikimedia.org/feed/
  2. Select "Transaction Logs" from the sidebar

Results:

Query (of class "ConpherenceTransactionQuery") overheated: examined more than 1,010 raw rows without finding 101 visible objects.

[2023-08-14 23:09:03] EXCEPTION: (Exception) Query (of class "ConpherenceTransactionQuery") overheated: examined more than 1,010 raw rows without finding 101 visible objects. at [<phabricator>/src/infrastructure/query/policy/PhabricatorPolicyAwareQuery.php:342]
 arcanist(), ava(), phabricator(), translations(), wmf-ext-misc()
   #0 <#2> PhabricatorPolicyAwareQuery::execute() called at [<phabricator>/src/applications/feed/query/PhabricatorFeedTransactionQuery.php:107]
   #1 <#2> PhabricatorFeedTransactionQuery::loadPage() called at [<phabricator>/src/infrastructure/query/policy/PhabricatorPolicyAwareQuery.php:251]
   #2 <#2> PhabricatorPolicyAwareQuery::execute() called at [<phabricator>/src/infrastructure/query/policy/PhabricatorCursorPagedPolicyAwareQuery.php:400]
   #3 <#2> PhabricatorCursorPagedPolicyAwareQuery::executeWithCursorPager(AphrontCursorPagerView) called at [<phabricator>/src/applications/search/engine/PhabricatorApplicationSearchEngine.php:1038]
   #4 <#2> PhabricatorApplicationSearchEngine::executeQuery(PhabricatorFeedTransactionQuery, AphrontCursorPagerView) called at [<phabricator>/src/applications/search/controller/PhabricatorApplicationSearchController.php:256]
   #5 <#2> PhabricatorApplicationSearchController::processSearchRequest() called at [<phabricator>/src/applications/search/controller/PhabricatorApplicationSearchController.php:91]
   #6 <#2> PhabricatorApplicationSearchController::processRequest() called at [<phabricator>/src/aphront/AphrontController.php:29]
   #7 <#2> AphrontController::handleRequest(AphrontRequest) called at [<phabricator>/src/aphront/AphrontController.php:71]
   #8 <#2> AphrontController::delegateToController(PhabricatorApplicationSearchController) called at [<phabricator>/src/applications/search/engine/PhabricatorApplicationSearchEngine.php:50]
   #9 <#2> PhabricatorApplicationSearchEngine::buildResponse() called at [<phabricator>/src/applications/feed/controller/PhabricatorFeedTransactionListController.php:13]
   #10 <#2> PhabricatorFeedTransactionListController::handleRequest(AphrontRequest) called at [<phabricator>/src/aphront/configuration/AphrontApplicationConfiguration.php:284]
   #11 phlog(Exception) called at [<phabricator>/src/aphront/handler/PhabricatorDefaultRequestExceptionHandler.php:41]
   #12 PhabricatorDefaultRequestExceptionHandler::handleRequestThrowable(AphrontRequest, Exception) called at [<phabricator>/src/aphront/configuration/AphrontApplicationConfiguration.php:751]
   #13 AphrontApplicationConfiguration::handleThrowable(Exception) called at [<phabricator>/src/aphront/configuration/AphrontApplicationConfiguration.php:296]
   #14 AphrontApplicationConfiguration::processRequest(AphrontRequest, PhutilDeferredLog, AphrontPHPHTTPSink, MultimeterControl) called at [<phabricator>/src/aphront/configuration/AphrontApplicationConfiguration.php:204]
   #15 AphrontApplicationConfiguration::runHTTPRequest(AphrontPHPHTTPSink) called at [<phabricator>/webroot/index.php:35]

See Also:

Note to myself: Easy to reproduce locally on /feed/transactions/ after creating 900 tasks and then mass-setting their view policy to a different user

Related Objects
Search...

Event Timeline

Aklapper created this task.Aug 15 2023, 10:28 AM
Aklapper triaged this task as Low priority.
Aklapper added a comment.Aug 15 2023, 11:27 AM

Created and set up a new custom default query not covering "All Transactions" (and thus not indexing Conpherence) in order to hide the problem: https://phabricator.wikimedia.org/feed/transactions/

Reedy moved this task from Backlog to To upstream/missing upstream link on the Upstream board.Aug 28 2023, 1:24 AM
Aklapper updated the task description. (Show Details)Sep 21 2023, 11:12 AM
Aklapper moved this task from Backlog to Upstreamed on the Phabricator (Upstream) board.
Aklapper moved this task from To upstream/missing upstream link to Reported Upstream on the Upstream board.
Aklapper merged a task: T271004: Cannot get list of repos via Conduit: "Query (of class \"PhabricatorRepositoryQuery\") overheated: examined more than 1,010 raw rows without finding 101 visible objects.".Nov 1 2023, 6:36 PM
Aklapper updated the task description. (Show Details)Nov 2 2023, 8:50 AM

I posted a potential approach to mitigate in P53126: do not query disabled applications.

P53126 T344232
1diff --git a/src/applications/conpherence/query/ConpherenceTransactionQuery.php b/src/applications/conpherence/query/ConpherenceTransactionQuery.php
2index d227217cb4..7c5ad2b23c 100644
3--- a/src/applications/conpherence/query/ConpherenceTransactionQuery.php
4+++ b/src/applications/conpherence/query/ConpherenceTransactionQuery.php
5@@ -7,4 +7,8 @@ final class ConpherenceTransactionQuery
6 return new ConpherenceTransaction();
7 }
8
9+ public function getApplicationClassName() {
10+ return 'PhabricatorConpherenceApplication';
11+ }
12+
13 }
14diff --git a/src/applications/feed/query/PhabricatorFeedTransactionQuery.php b/src/applications/feed/query/PhabricatorFeedTransactionQuery.php
15index d0a9f53e35..42ee332c1c 100644
16--- a/src/applications/feed/query/PhabricatorFeedTransactionQuery.php
17+++ b/src/applications/feed/query/PhabricatorFeedTransactionQuery.php
18@@ -155,6 +155,20 @@ final class PhabricatorFeedTransactionQuery
19 ->setAncestorClass('PhabricatorApplicationTransactionQuery')
20 ->execute();
21
22+ // Remove TransactionQuery classes that belong to previously installed apps
23+ // which got uninstalled in the meantime: Increases query performance and
24+ // decreases likeliness that a Query Overheated error will be displayed.
25+ // See also https://secure.phabricator.com/T13133 for more background.
26+ foreach ($queries as $key => $query) {
27+ if (method_exists($query, 'getApplicationClassName')) {
28+ $application_class_name = $query->getApplicationClassName();
29+ $application = new $application_class_name();
30+ if (!$application->isInstalled()) {
31+ unset($queries[$key]);
32+ }
33+ }
34+ }
35+
36 $type_map = array();
37
38 // If we're querying for specific transaction PHIDs, we only need to

Aklapper changed the task status from Open to Stalled.Jun 18 2025, 10:13 AM
Aklapper claimed this task.
Aklapper added a project: Release-Engineering-Team (Doing 😎).
Aklapper moved this task from Upstreamed to Solved upstream on the Phabricator (Upstream) board.
Aklapper added a subtask: T393840: Update to Phorge upstream rPca176821 / Arcanist upstream rARCd2d2afd0 (2025-09-08).
Aklapper moved this task from Reported Upstream to Patch merged upstream on the Upstream board.
Aklapper closed this task as Resolved.Sep 25 2025, 5:04 PM

This issue should now be fixed on phabricator.wikimedia.org after today's software deployment in T404134.

Aklapper closed subtask T393840: Update to Phorge upstream rPca176821 / Arcanist upstream rARCd2d2afd0 (2025-09-08) as Resolved.Sep 25 2025, 5:04 PM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits