[
https://issues.apache.org/jira/browse/CASSANDRA-10634?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14987869#comment-14987869
]
Joel Knighton commented on CASSANDRA-10634:
-------------------------------------------
I've pushed a fix that only performs datacenter-aware filtering when the
replication strategy is an instance of NetworkTopologyStrategy.
Dtests and unit tests are clean relative to cassandra-3.0 runs. Dtests
unfortunately show a lot of failures due to recent turbulence around the
upgrade test suite, so comparison is a bit laborious.
||cassandra-3.0||
|[dtests|http://cassci.datastax.com/job/jkni-10634-dtest/lastBuild/#showFailuresLink]|
|[unit tests|
http://cassci.datastax.com/view/Dev/view/jkni/job/jkni-10634-testall/1/]|
|[Cassandra
branch|https://github.com/jkni/cassandra/tree/e41bb463dd0f036ee79dd63a8235a9744ed09df6]|
I've added an additional dtest so that we test adding a datacenter with both
simple and network topology replication strategies. This dtest, along with the
dtest changes from [CASSANDRA-10621], is available
[here|https://github.com/jkni/cassandra-dtest/commits/mv-tests].
> Materialized Views filter paired endpoints to local DC even when not using
> DC-aware replication
> -----------------------------------------------------------------------------------------------
>
> Key: CASSANDRA-10634
> URL: https://issues.apache.org/jira/browse/CASSANDRA-10634
> Project: Cassandra
> Issue Type: Bug
> Components: Coordination
> Reporter: Joel Knighton
> Assignee: Joel Knighton
>
> When calculating paired view replicas on MV writes, we filter out base and
> view replicas that aren't in the local datacenter.
> If we are using SimpleStrategy, this filtering is incorrect, since our
> replica placement is not datacenter aware.
> We should behave the same as DC-aware consistency levels and only perform
> this datacenter-aware filtering if the replication strategy is an instance of
> NetworkTopologyStrategy.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
