Mailing-List: contact commits-help@cassandra.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cassandra.apache.org
Date: Thu, 16 Mar 2017 22:19:41 +0000 (UTC)
From: "Paulo Motta (JIRA)" <jira@apache.org>
To: commits@cassandra.apache.org
Message-ID: <JIRA.13034761.1484330595000.43219.1489702781760@Atlassian.JIRA>
In-Reply-To: <JIRA.13034761.1484330595000@Atlassian.JIRA>
References: <JIRA.13034761.1484330595000@Atlassian.JIRA> <JIRA.13034761.1484330595088@jira-lw-us.apache.org>
Subject: [jira] [Updated] (CASSANDRA-13121) repair progress message breaks
 legacy JMX support
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Thu, 16 Mar 2017 22:19:46 -0000


     [ https://issues.apache.org/jira/browse/CASSANDRA-13121?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Paulo Motta updated CASSANDRA-13121:
------------------------------------
    Labels: lhf  (was: )

> repair progress message breaks legacy JMX support
> -------------------------------------------------
>
>                 Key: CASSANDRA-13121
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-13121
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Streaming and Messaging
>            Reporter: Scott Bale
>            Priority: Minor
>              Labels: lhf
>
> The error progress message in {{RepairRunnable}} is not compliant with the {{LegacyJMXProgressSupport}} class, which uses a regex to match on the text of a progress event. Therefore, actual failures slip through as successes if using legacy JMX for repairs.
> In {{RepairRunnable}}
> {code}
>     protected void fireErrorAndComplete(String tag, int progressCount, int totalProgress, String message)
>     {
>         fireProgressEvent(tag, new ProgressEvent(ProgressEventType.ERROR, progressCount, totalProgress, message));
>         fireProgressEvent(tag, new ProgressEvent(ProgressEventType.COMPLETE, progressCount, totalProgress, String.format("Repair command #%d finished with error", cmd)));
>     }
> {code}
> Note the {{"Repair command #%d finished with error"}}
> See https://github.com/apache/cassandra/blob/trunk/src/java/org/apache/cassandra/repair/RepairRunnable.java#L109
> In {{LegacyJMXProgressSupport}}:
> {code}
>     protected static final Pattern SESSION_FAILED_MATCHER = Pattern.compile("Repair session .* for range .* failed with error .*");
>     protected static final Pattern SESSION_SUCCESS_MATCHER = Pattern.compile("Repair session .* for range .* finished");
> {code}
> See https://github.com/apache/cassandra/blob/trunk/src/java/org/apache/cassandra/utils/progress/jmx/LegacyJMXProgressSupport.java#L38
> Legacy JMX support was introduced for CASSANDRA-11430 (version 2.2.6) and the bug was introduced as part of CASSANDRA-12279 (version 2.2.8).


--
This message was sent by Atlassian JIRA
(v6.3.15#6346)