dcapwell commented on code in PR #3408:
URL: https://github.com/apache/cassandra/pull/3408#discussion_r1669369136


##########
test/simulator/test/org/apache/cassandra/simulator/test/SimulationTestBase.java:
##########
@@ -229,10 +230,16 @@ public static void 
simulate(IIsolatedExecutor.SerializableRunnable[] runnables,
         InstanceClassLoader classLoader = new InstanceClassLoader(1, 1, 
AbstractCluster.CURRENT_VERSION.classpath,
                                                                   
Thread.currentThread().getContextClassLoader(),
                                                                   
sharedClassPredicate,
-                                                                  new 
InterceptClasses((x) -> () -> 1.0f, (x) -> () -> 1.0f, 
NemesisFieldSelectors.get(), ClassLoader.getSystemClassLoader(), 
sharedClassPredicate.negate())::apply);
+                                                                  new 
InterceptClasses((x) -> () -> 1.0f, (x) -> () -> 1.0f,
+                                                                               
        NemesisFieldSelectors.get(),
+                                                                               
        ClassLoader.getSystemClassLoader(),
+                                                                               
        sharedClassPredicate.negate())::apply);
 
         ThreadGroup tg = new ThreadGroup("test");
-        InterceptorOfGlobalMethods interceptorOfGlobalMethods = new 
InterceptingGlobalMethods(NONE, null, failures, random);
+        InterceptedWait.CaptureSites.Capture capture = new 
InterceptedWait.CaptureSites.Capture(false, false, false);
+        InterceptorOfGlobalMethods interceptorOfGlobalMethods = 
IsolatedExecutor.transferAdhoc((IIsolatedExecutor.SerializableQuadFunction<InterceptedWait.CaptureSites.Capture,
 LongConsumer, Consumer<Throwable>, RandomSource, InterceptorOfGlobalMethods>) 
InterceptingGlobalMethods::new, classLoader)
+                                                                               
 .apply(capture, (ignore) -> {}, failures, random);

Review Comment:
   not familiar enough, can you explain this change?



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