maksaska commented on code in PR #11793:
URL: https://github.com/apache/ignite/pull/11793#discussion_r1928405563
##########
modules/core/src/test/java/org/apache/ignite/internal/processors/performancestatistics/PerformanceStatisticsThinClientTest.java:
##########
@@ -171,6 +178,87 @@ public void testCacheOperation() throws Exception {
checkCacheOperation(CACHE_GET_AND_REMOVE, cache ->
cache.getAndRemove(5));
}
+ /**
+ * Cache {@link TcpClientCache#putAllConflict} operation performed.
+ * @throws Exception If failed.
+ */
+ @Test
+ public void testCachePutAllConflict() throws Exception {
Review Comment:
I used the "cache" in naming to match the OperationType. e.g.
CACHE_PUT_ALL_CONFLICT
I agree, it can result in less code should we remove it, but I see that it's
used throughout the test class. I would prefer to follow that. WDYT?
--
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
