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

Yuki Morishita commented on CASSANDRA-9378:
-------------------------------------------

Initial patch here: https://github.com/yukim/cassandra/tree/9378

* Users can opt in logback metrics only by enabling it in logback.xml
* Little draw back for this approach is I had to remove access to Logger in 
CassandraMetrics, since instrumented appender is created when LoggerFactory is 
accessed. We want to configure metrics then initialize Logger. But this way we 
can catch metrics from the beginning.

> Instrument the logger with error count metrics by level
> -------------------------------------------------------
>
>                 Key: CASSANDRA-9378
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-9378
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core, Tools
>            Reporter: Jonathan Shook
>            Assignee: Yuki Morishita
>            Priority: Minor
>             Fix For: 2.2.x
>
>
> The ability to do sanity checks against logged errors and warning counts 
> could be helpful for several reasons. One of the most obvious would be as a 
> way to verify that no errors were logged during a (semi-) automated upgrade 
> or restart process.
> Fortunately, this is easy to enable as described here: 
> https://dropwizard.github.io/metrics/3.1.0/manual/logback/
> It was pointed out by [~jjordan] that this ability should exist in current 
> version if the user is willing to drop in the right jars and modify the 
> appender config.
> It would also be helpful as a programmatic feature with a toggle to enable or 
> disable, possibly with a cassandra.yaml config parameter. There may be some 
> users who would prefer to disable it to avoid calling another appender. If 
> testing shows the overhead for this to be sufficiently low, we could just 
> leave it on by default.
> These should be exposed via JMX when they are enabled.



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

Reply via email to