HomePhorge

Speed up DiffusionBrowseFileController by removing call to array_merge

Description

Speed up DiffusionBrowseFileController by removing call to array_merge

Summary: Some profiling using XHProf in the Dark Console showed me that Diffusion was wasting a ton of time on array_merge. This change sped up the loading of a large file in Diffusion from 16.8 seconds to 2.4 seconds.

Test Plan: Load files in Diffusion. They all look good. Also, use a PHP shell to try to manually verify that I still kinda remember some PHP and, yes, this is functionally equivalent to what was there before.

Reviewers: #blessed_reviewers, epriestley

Reviewed By: #blessed_reviewers, epriestley

Subscribers: epriestley, Korvin

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

Details

Provenance
Jacob HurwitzAuthored on May 9 2014, 6:06 PM
epriestleyCommitted on May 9 2014, 6:06 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rPe96c363eefe7: Add SMS support
Branches
Unknown
Tags
Unknown

Event Timeline