Homec4science

Stop doing an excessive amount of work in `diffusion.rawdiffquery`

Authored by epriestley <git@epriestley.com> on Sep 20 2016, 14:57.

Description

Stop doing an excessive amount of work in diffusion.rawdiffquery

Ref T11665.

Without -n 1, this logs the ENTIRE history of the repository. We
actually get the right result, but this is egregiously slow. Add -n 1
to return only one result.

It appears that I wrote this wrong way back in 2011, in D953. This
query is rarely used (until recently) which is likely why it has
escaped notice for so long.

Test Plan: Used Conduit console to execute diffusion.rawdiffquery.
Got the same results but spent 8ms instead of 200ms executing this
command, in a very small repository.

Details

Committed
epriestley <git@epriestley.com>Sep 20 2016, 15:00
Pushed
aubortJan 31 2017, 17:16
Parents
rPH03d323e9fdc2: Fix config-migration text for dashboard options
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH9329e6a12d9f: Stop doing an excessive amount of work in `diffusion.rawdiffquery` (authored by epriestley <git@epriestley.com>).Sep 20 2016, 15:00