Vladsz83 commented on code in PR #10485:
URL: https://github.com/apache/ignite/pull/10485#discussion_r1083943685


##########
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:
   > > This method calls GridCacheProcessor#clearStatistics().
   > 
   > How to do the same with public API?
   
   You always could clear cache statistics by calling Ignite cache API and by 
calling mx bean. Regarding to the IEP, we assume mx bean api pbsolete. Why 
using It again? Consider we wouldn't have to keep mxbean to support any spec 
like JCache spec. There would be no mxbean and no old mxbean#clear() method.



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