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

[cassandra] Add some more StorageProxy metrics #1393

Merged
merged 1 commit into from
Mar 2, 2015

Conversation

miketheman
Copy link
Contributor

This adds a few more node-wide metrics that complete the StorageProxy section of Cassandra's JMX metrics.
See here for more: http://wiki.apache.org/cassandra/JmxInterface#org.apache.cassandra.service.StorageProxy

@@ -40,7 +40,9 @@ init_config:
- MemtableSwitchCount
- MinRowSize
- PendingTasks
- RangeOpertaions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like a typo.
Also RangeOperation seems to be in the org.apache.cassandra.service domain, not org.apache.cassandra.db.

This adds a few more node-wide metrics that complete the `StorageProxy` section of Cassandra's JMX metrics.
See here for more: http://wiki.apache.org/cassandra/JmxInterface#org.apache.cassandra.service.StorageProxy
@miketheman miketheman force-pushed the miketheman/cass_storage_proxy branch from 1fa5c24 to 828ccec Compare March 1, 2015 23:54
@miketheman
Copy link
Contributor Author

There's no evidence of a typo other than your comment, so you just have traveled back in time or I've changed history. 😈

These tasks show up in the db domain using datadog-agent jmx list_everything, including the other items we already collect, like RecentRangeLatencyMicros.

@remh remh added this to the 5.3.0 milestone Mar 2, 2015
@remh
Copy link

remh commented Mar 2, 2015

Looks good after the typo fix 👻 .
Thanks!

remh pushed a commit that referenced this pull request Mar 2, 2015
[cassandra] Add some more StorageProxy metrics
@remh remh merged commit 3729b72 into master Mar 2, 2015
@remh remh deleted the miketheman/cass_storage_proxy branch March 23, 2015 17:23
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.

2 participants