Skip to content
This repository has been archived by the owner on Nov 20, 2019. It is now read-only.

Verspaetung should use the dropwizard-metrics library #17

Closed
rtyler opened this issue Jan 30, 2015 · 0 comments
Closed

Verspaetung should use the dropwizard-metrics library #17

rtyler opened this issue Jan 30, 2015 · 0 comments
Assignees
@rtyler
Labels
enhancement
Milestone
0.1.6 - Dropwizar...

Comments

@rtyler
Copy link
Member

rtyler commented Jan 30, 2015

After borrowing @mkristian's brain for some rubber duck debugging I'm realizing that it makes much more sense to look into using the Dropwziard Metrics library instead of hacking together java-statsd-client for my own uses.

Related to #7 and #11
@rtyler rtyler self-assigned this Jan 30, 2015
rtyler pushed a commit that referenced this issue Feb 6, 2015
Add a compile and runtime dependency on the metrics core and graphite…
f404289 
… code

References #17
rtyler pushed a commit that referenced this issue Feb 6, 2015
Major refactor to support dropwizard-metrics as the means of outputti…
7134759 
…ng metrics

This commit changes the structure of Verspaetung pretty dramatically to allow
for the registering of Gauges for the various offsets

With this change the KafkaPoller is pushing the latest offsets into a map and the
ZK consumer tree watchers are pushing consumer offsets into a separate map.

References #17
@rtyler rtyler mentioned this issue Feb 6, 2015
Closed
@rtyler rtyler closed this as completed Mar 20, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@rtyler rtyler

Labels
enhancement
Milestone
0.1.6 - Dropwizard like it's hot
Development

No branches or pull requests
1 participant
@rtyler