[jira] [Commented] (CASSANDRA-15163) Untangle RepairMessage sub-hierarchy of messages, use new messaging (more) correctly

["Aleksey Yeschenko (JIRA)" <ji...@apache.org>]

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

Aleksey Yeschenko commented on CASSANDRA-15163:
-----------------------------------------------

Branch [here|https://github.com/iamaleksey/cassandra/commits/15163-4.0], CI currently running [here|https://circleci.com/workflow-run/974762ef-c520-4c80-8fbf-f1e0c019c85e].

> Untangle RepairMessage sub-hierarchy of messages, use new messaging (more) correctly
> ------------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-15163
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-15163
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Messaging/Internode
>            Reporter: Aleksey Yeschenko
>            Priority: Normal
>
> There is currently a nested sub-hierarchy of messages used by repair - in {{RepairMessage}} - all sharing one verb, all sharing one deserialiser, all sharing one verb handler. The first two can and should be addressed, and it’s mostly a mechanical task to do so. The last one should be tackled separately, when we get to refactor repair some day (and it could definitely use some love).
> The proposed patch follows-up the work done in CASSANDRA-15066 and makes repair use the new messaging service in a more correct way, making verbs and deserialization more explicit. It wasn't included into the original commit so that to not make the single commit larger that it needed to be.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cassandra.apache.org
For additional commands, e-mail: commits-help@cassandra.apache.org