Github user aweisberg commented on a diff in the pull request:

    https://github.com/apache/cassandra-dtest/pull/39#discussion_r223397826
  
    --- 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 --
    
    Is this something we want to check for in all dtests all the time as part 
of checking the log for errors?
    
    It might help us avoid leaving spammy log lines that don't mean much to 
users in the long run.


---

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

Reply via email to