Page MenuHomePhabricator

Don't load unnecessary handle data on "transaction.search"
ClosedPublic

Authored by epriestley on Jan 30 2019, 8:20 PM.
Tags
None
Referenced Files
F12805589: D20068.id47950.diff
Wed, Mar 27, 5:22 PM
F12805587: D20068.id.diff
Wed, Mar 27, 5:22 PM
F12805586: D20068.diff
Wed, Mar 27, 5:22 PM
Unknown Object (File)
Sat, Mar 23, 12:35 PM
Unknown Object (File)
Sat, Mar 23, 12:35 PM
Unknown Object (File)
Sat, Mar 23, 12:35 PM
Unknown Object (File)
Sun, Mar 17, 3:19 AM
Unknown Object (File)
Feb 22 2024, 6:10 PM
Subscribers
None

Details

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.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

(My case for this and D20067 is a little weak since XHProf isn't running locally until I sort out T13232 by compiling Gentoo nine times, but I think they're almost certainly actual performance improvements and I just wanted to spend a few minutes on it and get rid of anything flagrantly silly.)

This revision is now accepted and ready to land.Feb 1 2019, 3:56 AM
This revision was automatically updated to reflect the committed changes.

In production against T13242, with ab -n 100 -c 1 ..., I get these before the change:

50th Percentile287ms
95th Percentile380ms

Deploying this now...

Now:

50th Percentile179ms
95th Percentile226ms

🐇⚡