dcapwell commented on code in PR #103:
URL: https://github.com/apache/cassandra-accord/pull/103#discussion_r1777742175
##########
accord-core/src/test/java/accord/burn/BurnTest.java:
##########
@@ -581,40 +579,22 @@ public void testOne()
private static void run(long seed)
{
Duration timeout = Duration.ofMinutes(3);
- Runnable fn = () -> run(seed, 1000);
- AsyncResult.Settable<?> promise = AsyncResults.settable();
- Thread t = new Thread(() -> {
Review Comment:
this timeout logic is now in `TimeoutUtils` as its copy/pasted in 2 places
and a 3rd needed it...
--
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]