Details
-
Improvement
-
Status: Resolved
-
Normal
-
Resolution: Fixed
-
None
-
Semantic
-
Low Hanging Fruit
-
All
-
None
-
Description
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.
Attachments
Issue Links
- causes
-
CASSANDRA-16469 replication_test.TestReplication.test_network_topology failure
- Resolved