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.

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

Add instrumentation for Guava caches #152

Closed
wants to merge 4 commits into from
Closed

Add instrumentation for Guava caches #152

wants to merge 4 commits into from

Conversation

celkins
Copy link

@celkins celkins commented Feb 3, 2012

No description provided.

@ntolia
Copy link

ntolia commented Feb 23, 2012

Is there any chance this patch will be accepted? We would appreciate it.

@codahale
Copy link
Contributor

One of the reasons I removed metrics-guava was because I didn't think it added enough value. We use Guava's cache functionality quite a bit, so I'm still interested in a solution, but this isn't it. I've added #167 to track that.

@codahale codahale closed this Feb 26, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants