cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Stefan Podkowinski (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CASSANDRA-12312) Restore JVM metric export for metric reporters
Date Wed, 03 Aug 2016 11:09:20 GMT

    [ https://issues.apache.org/jira/browse/CASSANDRA-12312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15405731#comment-15405731
] 

Stefan Podkowinski commented on CASSANDRA-12312:
------------------------------------------------

I've now included documentation on the added metrics in the trunk patch. 

[~tjake], what do you think about including metrics on thread states based on [ThreadStatesGaugeSet.java|https://github.com/dropwizard/metrics/blob/3.1-maintenance/metrics-jvm/src/main/java/com/codahale/metrics/jvm/ThreadStatesGaugeSet.java]?
I haven't included them so far as I'm not really sure if it makes sense to report these numbers
as part of a performance monitoring, except maybe the number of runnable threads. But I wonder
if it's really worth to constantly poll these values and how expensive that would be. 

> Restore JVM metric export for metric reporters
> ----------------------------------------------
>
>                 Key: CASSANDRA-12312
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-12312
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Stefan Podkowinski
>            Assignee: Stefan Podkowinski
>              Labels: lhf
>             Fix For: 2.2.x, 3.0.x, 3.x
>
>         Attachments: 12312-2.2.patch, 12312-3.0.patch, 12312-trunk.patch, metrics-jvm-3.1.0.jar.asc
>
>
> JVM instrumentation as part of dropwizard metrics has been moved to a separate {{metrics-jvm}}
artifact in metrics-v3.0. After CASSANDRA-5657, no jvm related metrics will be exported to
any reporter configured via {{metrics-reporter-config}}, as this isn't part of {{metrics-core}}
anymore. As memory and GC stats are essential for monitoring Cassandra, this turns out to
be a blocker for us for upgrading to 2.2.
> I've included a patch that would add the now separate {{metrics-jvm}} package and enables
some of the provided metrics on startup in case a metrics reporter is used ({{-Dcassandra.metricsReporterConfigFile}}).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message