Skip to content

Reduce client logger overhead#960

Merged
gigabo merged 6 commits into
redfin:masterfrom
gigabo:client-logger-perf
Oct 17, 2017
Merged

Reduce client logger overhead#960
gigabo merged 6 commits into
redfin:masterfrom
gigabo:client-logger-perf

Conversation

@gigabo

@gigabo gigabo commented Oct 10, 2017

Copy link
Copy Markdown
Contributor

Small client-side performance improvement for sites that log a lot.

Added a benchmark to measure the impact.

Before:

info no transports      0.0030393327980000003
info noop transport     0.007346843629917184
time no transports      0.004063845286219832
time noop transport     0.008934361976376487

After:

info no transports      0.00004931556964542886
info noop transport     0.001289028179448941
time no transports      0.00010602540850444264
time noop transport     0.0011963334522868431
@gigabo gigabo merged commit 84d1ad4 into redfin:master Oct 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants

@gigabo @sresant @davidk107