[ https://issues.apache.org/jira/browse/CASSANDRA-14759?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16625832#comment-16625832 ]
Alex Petrov commented on CASSANDRA-14759: ----------------------------------------- +1 as logic looks right, just two minor things for discussion: * should we get rid of generics {{<E extends Endpoints<E>> E}} there since we only have token writes and these methods are called with {{EndpointsForToken}} only? * semantically, wouldn't it be better to leave full pending in pending and filter natural transient out? It might be slightly confusing that natural replica flips from transient to full: it doesn't seem as we have inherent limitations that prevent us from that, or? > Transient->Full movements mishandle consistency level upgrade > ------------------------------------------------------------- > > Key: CASSANDRA-14759 > URL: https://issues.apache.org/jira/browse/CASSANDRA-14759 > Project: Cassandra > Issue Type: Bug > Components: Core > Reporter: Benedict > Assignee: Benedict > Priority: Major > Labels: Availability, correctness, transient-replication > Fix For: 4.0 > > > While we need treat a transitioning node as ‘full’ for writes, so that it can > safely begin serving full data requests once it has finished, we cannot > maintain it in the ‘pending’ collection else we will also increase our > consistency requirements by a node that doesn’t exist. -- This message was sent by Atlassian JIRA (v7.6.3#76005) --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org