nizhikov commented on code in PR #10485:
URL: https://github.com/apache/ignite/pull/10485#discussion_r1083757319
##########
modules/clients/src/test/java/org/apache/ignite/internal/processors/rest/RestMemcacheProtocolSelfTest.java:
##########
@@ -185,8 +185,8 @@ public void testReplace() throws Exception {
*/
@Test
public void testMetrics() throws Exception {
- grid().cache(DEFAULT_CACHE_NAME).localMxBean().clear();
- grid().cache(CACHE_NAME).localMxBean().clear();
+ grid().cache(DEFAULT_CACHE_NAME).clearStatistics();
Review Comment:
Why do we need new method `clearStatistics`?
AFAICU clear method comes from `javax.cache.management.CacheStatisticsMXBean`
Can we just invoke method from MX bean instance as we do now?
--
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]