Skip to content
This repository has been archived by the owner on Mar 9, 2022. It is now read-only.

Fix HTTP listener benchmarks & bump perfcheck reference #81

Merged
merged 2 commits into from
May 29, 2018

Conversation

mjs
Copy link
Contributor

@mjs mjs commented May 28, 2018

The interaction of the last two PRs resulted in the HTTP listener benchmark logs being broken. This change fixes that as well as suppressing log output during these benchmarks.

The default reference revision used by perfcheck has been bumped so that log output is suppressed during the perfcheck reference benchmarks. This ensures that benchcheck will see all available benchmark output.

@mjs mjs requested a review from oplehto May 28, 2018 21:54
mjs added 2 commits May 29, 2018 11:25
The interaction of the last two PRs resulted in the HTTP listener
tests and benchmarks being broken. This change fixes that as well as
suppressing log output during the HTTP listener benchmarks.
This means log output is now suppressed during the perfcheck reference
benchmarks.
@mjs mjs force-pushed the bump-perfcheck-reference branch from 8e9a712 to 22b95b7 Compare May 28, 2018 23:25
This was referenced May 29, 2018
@mjs mjs requested a review from amnonbc May 29, 2018 04:03
@mjs mjs merged commit 3616d83 into jumptrading:master May 29, 2018
@mjs mjs deleted the bump-perfcheck-reference branch May 29, 2018 21:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants