frankgh commented on code in PR #2318:
URL: https://github.com/apache/cassandra/pull/2318#discussion_r1202633170


##########
src/java/org/apache/cassandra/utils/MBeanWrapper.java:
##########
@@ -119,17 +138,23 @@ static ObjectName create(String mbeanName, OnException 
onException)
         }
     }
 
-    static class NoOpMBeanWrapper implements MBeanWrapper
+    Set<ObjectName> queryNames(ObjectName name, QueryExp query);
+
+    MBeanServer getMbs();
+
+    class NoOpMBeanWrapper implements MBeanWrapper
     {
         public void registerMBean(Object obj, ObjectName mbeanName, 
OnException onException) {}
         public void registerMBean(Object obj, String mbeanName, OnException 
onException) {}
         public boolean isRegistered(ObjectName mbeanName, OnException 
onException) { return false; }
         public boolean isRegistered(String mbeanName, OnException onException) 
{ return false; }
         public void unregisterMBean(ObjectName mbeanName, OnException 
onException) {}
         public void unregisterMBean(String mbeanName, OnException onException) 
{}
+        public Set<ObjectName> queryNames(ObjectName name, QueryExp query) 
{return Collections.emptySet(); }
+        public MBeanServer getMbs() {return null; }

Review Comment:
   ```suggestion
           public MBeanServer getMbs() { return null; }
   ```



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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to