Homestyx hydra
Diffusion hydra f1a588c771f6

Add a basic profiler to Herald transcripts

Description

Add a basic profiler to Herald transcripts

Summary:
Ref T13298. Add a simple profiler as a starting point to catch any egregiously expensive rules or conditions.

This doesn't profile rule actions, so if "Add subscriber" (or whatever) is outrageously expensive it won't show up on the profile. Right now, actions get evaluated inside the Adapter so they're hard to profile. A future change could likely dig them out without too much trouble. I generally expect actions to be less expensive than conditions.

This also can't pin down a specific condition being expensive, but if you see that H123 takes 20s to evaluate you can probably guess that the giant complicated regex is the expensive part.

Test Plan: {F6473407}

Reviewers: amckinley

Reviewed By: amckinley

Maniphest Tasks: T13298

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

Details

Provenance
epriestleyAuthored on May 31 2019, 10:22 AM
sirocylPushed on Oct 16 2024, 5:49 AM
Parents
R1:b7aacaa4d30b: Differentiate Remarkup header sizes more clearly
Branches
Unknown
Tags
Unknown

Event Timeline