Homestyx hydra
Diffusion hydra 50ed42761ce9

Don't issue a bazillion queries to load Differential object lists

Description

Don't issue a bazillion queries to load Differential object lists

Summary:
Ref T3496. Currently, we call loadAssets() on each revision table, which invokes a new revision query and a pile of subqueries.

Instead, add needFlags() and needDrafts() to RevisionQuery. Some day these could perhaps be more generic.

Test Plan:

  • Viewed home, differential, etc., no longer saw 9203809238 queries being run for no reason.
  • Drafts and flags still appear properly.

Reviewers: btrahan

Reviewed By: btrahan

CC: aran

Maniphest Tasks: T3496

Differential Revision: https://secure.phabricator.com/D8277

Details

Provenance
epriestleyAuthored on Feb 18 2014, 8:57 PM
sirocylPushed on Oct 16 2024, 5:49 AM
Parents
R1:65bc2b1ac53d: Implement "Pro" version of revision editor, with one field
Branches
Unknown
Tags
Unknown

Event Timeline