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

Removed duplicate evictions metric #2577

Merged
merged 1 commit into from
Mar 28, 2017
Merged

Removed duplicate evictions metric #2577

merged 1 commit into from
Mar 28, 2017

Conversation

mgresser
Copy link
Contributor

@theonajim noticed I duplicated evictions in #412

This fixes that.

Required for all PRs:

  • CHANGELOG.md updated (we recommend not updating this until the PR has been approved by a maintainer)
  • [ x ] Sign CLA (if not already signed)
  • [ x ] README.md updated (if adding a new plugin)

@theonajim
Copy link

@mgresser thanks! I was also wondering if the following stats should be published from memcached as well.

cas_badval
cmd_flush
cmd_touch
evicted_unfetched
expired_unfetched
reclaimed
touch_hits
touch_misses

@danielnelson danielnelson merged commit 1100a98 into influxdata:master Mar 28, 2017
@danielnelson
Copy link
Contributor

@theonajim Can you open an issue for these other stats?

calerogers pushed a commit to calerogers/telegraf that referenced this pull request Apr 5, 2017
vlamug pushed a commit to vlamug/telegraf that referenced this pull request May 30, 2017
maxunt pushed a commit that referenced this pull request Jun 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants