grighetto commented on a change in pull request #5:
URL: https://github.com/apache/cassandra-harry/pull/5#discussion_r538849169
##########
File path: harry-runner/src/harry/runner/HarryRunner.java
##########
@@ -36,21 +38,13 @@
import harry.corruptor.QueryResponseCorruptor;
import harry.model.ExhaustiveChecker;
import harry.model.OpSelectors;
-import harry.model.sut.InJvmSut;
-import org.apache.cassandra.distributed.impl.IsolatedExecutor;
-import org.apache.cassandra.distributed.test.TestBaseImpl;
-public class HarryRunner extends TestBaseImpl
+public interface HarryRunner
Review comment:
I turned this into an interface with a default implementation that's
further implemented by both `HarryRunnerJvm` and `HarryRunnerExternal` and
removed the dependency on `TestBaseImpl`.
----------------------------------------------------------------
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
