[CASSANDRA-13713] Move processing of EchoMessage response to gossip stage

• Type: Bug
• Status: Resolved
• Priority: Low
• Resolution: Fixed
• Fix Version/s: 4.0-alpha1, 4.0
• Component/s: Legacy/Distributed Metadata
• Severity: Low
• Assignee: Jason Brown
• Reporter: Jason Brown

Authors

Jason Brown

Reviewers

Joel Knighton

• Created: 20/Jul/17 21:10
• Updated: 15/May/20 08:06
• Resolved: 29/Mar/18 12:43

Description

Currently, when a node receives an EchoMessage, is sends a simple ACK reply back (see EchoVerbHandler). The ACK is sent on the small message connection, and because it is 'generically' typed as Verb.REQUEST_RESPONSE, is consumed on a Stage.REQUEST_RESPONSE thread. The proper thread for this response to be consumed is Stage.GOSSIP, that way we can move more of the updating of the gossip state to a single, centralized thread, and less abuse of gossip's shared mutable state can occur.