To be investigated.
Description
Details
Related Objects
- Mentioned In
- T386704: More AVA performance improvements
rPHDEPebe3a947ad22: update submodules for 2024-06-18 release
rPHAVA69ed4ebbb9e3: Limit querying latest user transactions in Maniphest to recent IDs
rPHAVAd177aa30a21c: AVA: Check earlier if acting user is admin
rPHAVA65efd2818f96: Count user transactions in Maniphest only in last two million rows
Event Timeline
Change #1039766 had a related patch set uploaded (by Aklapper; author: Aklapper):
[phabricator/antivandalism@wmf/stable] AVA: Check earlier if acting user is admin
Change #1039786 had a related patch set uploaded (by Aklapper; author: Aklapper):
[phabricator/antivandalism@wmf/stable] Count user transactions in Maniphest only in last two million rows
Regarding https://gerrit.wikimedia.org/r/1039786 , the SQL query for my account took on average approx. 1.95 seconds on a warm cache in production. After applying the patch, querying only the last two million rows instead of approx all 10 million rows, it takes on average approx. 0.54 seconds.
Change #1039791 had a related patch set uploaded (by Aklapper; author: Aklapper):
[phabricator/antivandalism@wmf/stable] Limit querying latest user transactions in Maniphest to recent IDs
Regarding https://gerrit.wikimedia.org/r/1039791 , the SQL query for my account for the last 24hours (arbitrary value chosen for $hours, real value depends on the config of the Phabricator instance) went down from average 2.07 seconds to 0.17 seconds after restricting the query subset to the last half a million transactions.
Change #1039766 merged by Brennen Bearnes:
[phabricator/antivandalism@wmf/stable] AVA: Check earlier if acting user is admin
Change #1039786 merged by Brennen Bearnes:
[phabricator/antivandalism@wmf/stable] Count user transactions in Maniphest only in last two million rows
Change #1039791 merged by Brennen Bearnes:
[phabricator/antivandalism@wmf/stable] Limit querying latest user transactions in Maniphest to recent IDs