maksaska commented on code in PR #11793:
URL: https://github.com/apache/ignite/pull/11793#discussion_r1948066246


##########
modules/core/src/test/java/org/apache/ignite/internal/processors/performancestatistics/PerformanceStatisticsThinClientTest.java:
##########
@@ -96,7 +119,19 @@ public class PerformanceStatisticsThinClientTest extends 
AbstractPerformanceStat
     @Override protected void afterTestsStopped() throws Exception {
         super.afterTestsStopped();
 
-        thinClient.close();
+        stopAllGrids();
+    }
+
+    /** {@inheritDoc} */
+    @Override protected void beforeTest() throws Exception {
+        thinClient.getOrCreateCache(new ClientCacheConfiguration()

Review Comment:
   Fixed it: Now createCache() is used to start the cache with updated 
configuration. After each test the cache is destroyed 



##########
modules/core/src/test/java/org/apache/ignite/internal/processors/performancestatistics/PerformanceStatisticsThinClientTest.java:
##########
@@ -66,12 +81,20 @@ public class PerformanceStatisticsThinClientTest extends 
AbstractPerformanceStat
     /** Thin client. */
     private static IgniteClient thinClient;
 
+    /** */
+    @Parameterized.Parameter
+    public CacheAtomicityMode atomicityMode;
+
+    /** */
+    @Parameterized.Parameters(name = "atomicity={0}")

Review Comment:
   fixed



-- 
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: notifications-unsubscr...@ignite.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to