Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

optimize metric reporting #40

Merged
merged 13 commits into from
Aug 13, 2015
Merged

optimize metric reporting #40

merged 13 commits into from
Aug 13, 2015

Conversation

afex
Copy link
Owner

@afex afex commented Aug 12, 2015

  • sends data to collectors in parallel
  • speeds up the statsd collector by using buffered client

includes a load testing harness to aid investigation of performance changes

afex added 13 commits July 31, 2015 16:26
@afex
fix double counting stats, buffer updates channel
e098e75
@afex
begin work on load test driver. simple http server with a circuit rep…
a9ec81e 
…orting to statsd
@afex
block on command execution. rotate delay to see circuits opening and …
6a68a4c 
…closing
@afex
Merge branch 'fix-double-counting-stats' into loadtest
50b1477
@afex
Merge branch 'master' into loadtest
0bac5a7
@afex
send request wall time to statsd to measure user-facing latency
048338c
@afex
increment metric collectors in parallel to speed up processing of Upd…
81a5c0b 
…ates channel and reduce total duration.
@afex
minimize total duration by using unbuffered channel for updates. reli…
42c25ea 
…es on new, faster metrics consumption
@afex
Merge branch 'master' into loadtest
be7b59e
@afex
use buffered client for statsd collector to reduce TotalDuration over…
5b79165 
…head
@afex
fmt
34ebd4c
@afex
have test account for buffered updates channel
2f41336
@afex
have test account for buffered updates channel
d68c5e7
afex added a commit that referenced this pull request Aug 13, 2015
@afex
Merge pull request #40 from afex/loadtest
648a34a 
optimize metric reporting
@afex afex merged commit 648a34a into master Aug 13, 2015
@afex afex deleted the loadtest branch August 13, 2015 18:48
@afex afex mentioned this pull request Aug 14, 2015
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@afex