This is an automated email from the ASF dual-hosted git repository.

marcuse pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/cassandra.git


The following commit(s) were added to refs/heads/trunk by this push:
     new 86a353b443 Wait for move to finish before cleanup in 
TransientRangeMovement2Test#testMoveForward
86a353b443 is described below

commit 86a353b4436ee5d6794d4968dfde30ed6f791030
Author: Marcus Eriksson <[email protected]>
AuthorDate: Wed Jul 31 15:33:07 2024 +0200

    Wait for move to finish before cleanup in 
TransientRangeMovement2Test#testMoveForward
    
    Patch by marcuse; reviewed by Sam Tunnicliffe for CASSANDRA-19803
---
 .../cassandra/distributed/test/TransientRangeMovement2Test.java      | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git 
a/test/distributed/org/apache/cassandra/distributed/test/TransientRangeMovement2Test.java
 
b/test/distributed/org/apache/cassandra/distributed/test/TransientRangeMovement2Test.java
index 2a4898884e..c52b8920c4 100644
--- 
a/test/distributed/org/apache/cassandra/distributed/test/TransientRangeMovement2Test.java
+++ 
b/test/distributed/org/apache/cassandra/distributed/test/TransientRangeMovement2Test.java
@@ -118,7 +118,8 @@ public class TransientRangeMovement2Test extends 
TestBaseImpl
             // pausing + cleanup in.
             Callable<Epoch> pending = pauseBeforeCommit(cluster.get(1), (e) -> 
e instanceof PrepareMove.MidMove);
 
-            new Thread(() -> cluster.get(1).nodetoolResult("move", 
"15").asserts().success()).start();
+            Thread t = new Thread(() -> cluster.get(1).nodetoolResult("move", 
"15").asserts().success());
+            t.start();
             Epoch pauseBeforeEnacting = pending.call().nextEpoch();
 
             Callable<?> beforeEnacted = pauseBeforeEnacting(cluster.get(3), 
pauseBeforeEnacting);
@@ -130,7 +131,7 @@ public class TransientRangeMovement2Test extends 
TestBaseImpl
 
             unpauseEnactment(cluster.get(3));
             waitForCMSToQuiesce(cluster, cluster.get(1));
-
+            t.join();
             cluster.forEach(i -> 
i.nodetoolResult("cleanup").asserts().success());
             assertAllContained(localStrs(cluster.get(1)),
                                newArrayList("22", "24", "26", "28", "30"),


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

Reply via email to