Github user jasobrown commented on a diff in the pull request:
https://github.com/apache/cassandra-dtest/pull/15#discussion_r159234036
--- Diff: byteman/4.0/decommission_failure_inject.btm ---
@@ -10,8 +10,8 @@ METHOD prepareSynAck
AT INVOKE startStreamingFiles
BIND peer = $0.peer
# set flag to only run this rule once.
-IF peer.equals(InetAddress.getByName("127.0.0.1")) AND NOT flagged("done")
+IF
peer.equals(org.apache.cassandra.locator.InetAddressAndPort.getByName("127.0.0.1"))
AND NOT flagged("done")
--- End diff --
trivial nit: is the fully qualified class name necessary? or can you get
away with just the class name?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org
For additional commands, e-mail: pr-h...@cassandra.apache.org
