[GitHub] [cassandra-dtest] maedhroz commented on a change in pull request #59: Add tests for CASSANDRA-8272 and CASSANDRA-8273

[GitBox <gi...@apache.org>]

maedhroz commented on a change in pull request #59:
URL: https://github.com/apache/cassandra-dtest/pull/59#discussion_r430858206



##########
File path: replica_side_filtering_test.py
##########
@@ -495,4 +509,26 @@ class TestAllowFiltering(ReplicaSideFiltering):
     __test__ = True
 
     def create_index(self):
-        return False
\ No newline at end of file
+        return False
+
+    @since('4.0')
+    def test_update_missed_by_transient_replica(self):
+        self._prepare_cluster(
+            create_table="CREATE TABLE t (k int PRIMARY KEY, v text)",
+            both_nodes=["INSERT INTO t(k, v) VALUES (0, 'old')"],
+            only_node1=["UPDATE t SET v = 'new' WHERE k = 0"],
+            transient=True)
+
+        self._assert_none("SELECT * FROM t WHERE v = 'old'")

Review comment:
       @adelapena Just to make sure we're on the same page, at `ALL` it is this first query that would fail without RFP, whereas the second query would be fine, correct?




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@cassandra.apache.org
For additional commands, e-mail: pr-help@cassandra.apache.org