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

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

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



##########
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:
       That's correct. The second assertion verifies the case where the RFP extra query returns the old value, which is discarded in the second merge round.




----------------------------------------------------------------
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