cassandra-pr mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ifesdjeen <...@git.apache.org>
Subject [GitHub] cassandra-dtest pull request #39: Add no-read assert to read-repair test
Date Tue, 09 Oct 2018 14:50:40 GMT
Github user ifesdjeen commented on a diff in the pull request:

    https://github.com/apache/cassandra-dtest/pull/39#discussion_r223734988
  
    --- Diff: read_repair_test.py ---
    @@ -495,6 +495,9 @@ def test_normal_read_repair(self):
                 assert storage_proxy.speculated_rr_read == 0
                 assert storage_proxy.speculated_rr_write == 0
     
    +        warn = node2.grep_log("Message-to-self TYPE:READ")
    --- End diff --
    
    Ultimately - yes. But unfortunately there are still a couple of places we still do that.
Maybe we should widen the scope of the ticket and check more places where this is happening.
Or open a separate issue.


---

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


Mime
View raw message