Fixes #10442 - Reduce verbosity when JMX finds overloaded setter

Now the log statement is only emitted when there is no matching setter method, but a mismatched one exists.

Signed-off-by: Simone Bordet <[email protected]>

jetty-core/jetty-jmx/src/main/java/org/eclipse/jetty/jmx/MetaData.java

@@ -445,25 +445,24 @@ private Method findSetter(ManagedAttribute attribute, Method getter, String name
                 setterName = "set" + name.substring(0, 1).toUpperCase(Locale.ENGLISH) + name.substring(1);
 
             Method setter = null;
+            Method candidate = null;
             Class<?> klass = getter.getDeclaringClass();
             for (Method method : klass.getMethods())
             {
                 if (method.getName().equals(setterName) && method.getParameterCount() == 1)
                 {
-                    if (setter != null)
+                    candidate = method;
+                    if (getter.getReturnType().equals(method.getParameterTypes()[0]))
                     {
-                        LOG.info("Multiple setters for mbean attribute {} in {}", name, klass);
-                        continue;
-                    }
-                    if (!getter.getReturnType().equals(method.getParameterTypes()[0]))
-                    {
-                        LOG.info("Getter/setter type mismatch for mbean attribute {} in {}", name, klass);
-                        continue;
+                        setter = method;
+                        break;
                     }
-                    setter = method;
                 }
             }
 
+            if (candidate != null && setter == null)
+                LOG.info("Getter/setter type mismatch for mbean attribute {} in {}, attribute will be read-only", name, klass);
+
             return setter;
         }