smiklosovic commented on code in PR #221:
URL: https://github.com/apache/cassandra-dtest/pull/221#discussion_r1233239443


##########
topology_test.py:
##########
@@ -443,6 +443,8 @@ def test_crash_during_decommission(self):
         cluster.populate(3).start()
 
         node1, node2 = cluster.nodelist()[0:2]
+        session = self.patient_cql_connection(node1)
+        session.execute("ALTER KEYSPACE system_distributed WITH REPLICATION = 
{'class':'SimpleStrategy', 'replication_factor':'2'};")

Review Comment:
   this is in other test methods too, without this, we are actually not 
decommissioning anything because decommission will fail as there will not be 
enough replicas if replication factor will be 3 and we want to decommission to 
have just 2. We need to lower this to 2 so by decommissioning we will have 
still enough.



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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to