You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Aleksey Yeschenko (JIRA)" <ji...@apache.org> on 2015/01/23 02:04:08 UTC
[jira] [Commented] (CASSANDRA-5657) remove deprecated metrics
[ https://issues.apache.org/jira/browse/CASSANDRA-5657?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14288523#comment-14288523 ]
Aleksey Yeschenko commented on CASSANDRA-5657:
----------------------------------------------
Attached nits.txt with some nits and issues fixed:
- fixed the mixup in SEP#newExecutor() argument order - StageManager#multiThreadedLowSignalStage() was calling them in the wrong order otherwise
- removed unused fields in LatencyMetrics
- replaced silly looking Stage#isJmxHidden()
Otherwise the branch mostly LGTM, +1.
P.S. There are many imports in wrong order, redundant @Override-s when implementing interface/abstract methods (I assume automatically generated), and else/catch on the same line as the closing bracket. Ideally those should be fixed on commit.
> remove deprecated metrics
> -------------------------
>
> Key: CASSANDRA-5657
> URL: https://issues.apache.org/jira/browse/CASSANDRA-5657
> Project: Cassandra
> Issue Type: Task
> Components: Tools
> Reporter: Jonathan Ellis
> Assignee: T Jake Luciani
> Labels: technical_debt
> Fix For: 3.0
>
> Attachments: 5657.txt, 5657v2.txt, nits.txt
>
>
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)