oleg-vlsk commented on code in PR #11793:
URL: https://github.com/apache/ignite/pull/11793#discussion_r1912379134


##########
modules/core/src/test/java/org/apache/ignite/internal/processors/performancestatistics/PerformanceStatisticsThinClientTest.java:
##########
@@ -199,6 +210,36 @@ private void checkCacheOperation(OperationType op, 
Consumer<ClientCache<Object,
         assertEquals(1, ops.get());
     }
 
+    /**
+     * Cache {@link TcpClientCache#putAllConflict} operation perfomed
+     * @param map {@link Map} with entries for cache put all.
+     * @return cache {@link Consumer<ClientCache>}.
+     */
+    private Consumer<ClientCache<Object, Object>> putAllConflict(Map<Integer, 
Object> map) {
+        Map<Integer, T3<Object, GridCacheVersion, Long>> drMap = new 
HashMap<>();
+
+        GridCacheVersion confl = new GridCacheVersion(1, 0, 1, (byte)2);
+
+        map.forEach((key, value) -> drMap.put(key, new T3<>(value, confl, 
CU.EXPIRE_TIME_ETERNAL)));

Review Comment:
   Using forEach on a singleton map seems unnecessary, maybe something like
   ```
   Map<Integer, T3<Object, GridCacheVersion, Long>> drMap1 = F.asMap(
               F.firstEntry(map).getKey(),
               new T3<>(F.firstValue(map), confl, CU.EXPIRE_TIME_ETERNAL));
   ```
   Again, do we really need a map here as an argument?



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