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.

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

Functions to retrieve registered metrics #102

Merged
merged 1 commit into from Apr 11, 2016

Conversation

Projects
None yet
2 participants
@sunng87
Contributor

sunng87 commented Apr 11, 2016

Hi @sjl , in our application, the names of metrics were unknown at compile time. So I need some functions to get all metrics from registry by type. This patch added functions for this requirement.

(feat) added functions to retrieve different type of metrics from reg…
…istry

Signed-off-by: Ning Sun <sunng@about.me>

@michaelklishin michaelklishin merged commit e427676 into metrics-clojure:master Apr 11, 2016

@michaelklishin michaelklishin self-assigned this Apr 11, 2016

@michaelklishin

This comment has been minimized.

Show comment
Hide comment
@michaelklishin

michaelklishin Apr 11, 2016

Collaborator

Thank you.

Collaborator

michaelklishin commented Apr 11, 2016

Thank you.

@sunng87 sunng87 deleted the sunng87:feature/metric-reader branch Apr 11, 2016

@sunng87

This comment has been minimized.

Show comment
Hide comment
@sunng87

sunng87 Apr 12, 2016

Contributor

Thanks you @michaelklishin , is it possible to create a release for this?

Contributor

sunng87 commented Apr 12, 2016

Thanks you @michaelklishin , is it possible to create a release for this?

@michaelklishin

This comment has been minimized.

Show comment
Hide comment
@michaelklishin

michaelklishin Apr 12, 2016

Collaborator

@sunng87 yes, should be up within 24 hours.

Collaborator

michaelklishin commented Apr 12, 2016

@sunng87 yes, should be up within 24 hours.

@sunng87

This comment has been minimized.

Show comment
Hide comment
@sunng87
Contributor

sunng87 commented Apr 13, 2016

@michaelklishin

This comment has been minimized.

Show comment
Hide comment
@michaelklishin

michaelklishin Apr 13, 2016

Collaborator

Looks like it will take me another day or two. Apologies.

Collaborator

michaelklishin commented Apr 13, 2016

Looks like it will take me another day or two. Apologies.

@sunng87

This comment has been minimized.

Show comment
Hide comment
@sunng87

sunng87 Apr 13, 2016

Contributor

Never mind. I can use java method directly for now.

Contributor

sunng87 commented Apr 13, 2016

Never mind. I can use java method directly for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment