cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "T Jake Luciani (JIRA)" <>
Subject [jira] [Commented] (CASSANDRA-10413) Replaying materialized view updates from commitlog after node decommission crashes Cassandra
Date Wed, 30 Sep 2015 13:48:04 GMT


T Jake Luciani commented on CASSANDRA-10413:

bq. I'm not yet sure that selecting the local address as the view replica is always appropriate
in this situation

I agree, it seems like if the node is not joined we have no clue if the gossip state is stale
or not. In which case we should always apply the batchlog.

So rather than do this check in getViewNaturalEndpoint() add the gossip state check directly
in mutateMV and force it through the batchlog only.  We don't even need to do the async updates
since they will fail also.

> Replaying materialized view updates from commitlog after node decommission crashes Cassandra
> --------------------------------------------------------------------------------------------
>                 Key: CASSANDRA-10413
>                 URL:
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Joel Knighton
>            Assignee: T Jake Luciani
>            Priority: Critical
>             Fix For: 3.0.0 rc2
>         Attachments: n1.log, n2.log, n3.log, n4.log, n5.log
> This issue is reproducible through a Jepsen test, runnable as
> {code}
> lein with-profile +trunk test :only
> {code}
> This test crashes/restarts nodes while decommissioning nodes. These actions are not coordinated.
> In [10164|], we introduced a change
to re-apply materialized view updates on commitlog replay.
> Some nodes, upon restart, will crash in commitlog replay. They throw the "Trying to get
the view natural endpoint on a non-data replica" runtime exception in getViewNaturalEndpoint.
I added logging to getViewNaturalEndpoint to show the results of replicationStrategy.getNaturalEndpoints
for the baseToken and viewToken.
> It can be seen that these problems occur when the baseEndpoints and viewEndpoints are
identical but do not contain the broadcast address of the local node.
> For example, a node at crashes on replay of a write whose base token and view
token replicas are both [,,]. It seems we try to guard against this
by considering pendingEndpoints for the viewToken, but this does not appear to be sufficient.
> I've attached the system.logs for a test run with added logging. In the attached logs,
n1 is at, n2 is at, and so on. is the decommissioned node.

This message was sent by Atlassian JIRA

View raw message