Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Reduce verbosity when JMX finds overloaded setter #10442

Closed
sbordet opened this issue Aug 31, 2023 · 1 comment · Fixed by #10443
Closed

Reduce verbosity when JMX finds overloaded setter #10442

sbordet opened this issue Aug 31, 2023 · 1 comment · Fixed by #10443
Assignees
Labels
Bug For general bugs on Jetty side

Comments

@sbordet
Copy link
Contributor

sbordet commented Aug 31, 2023

Jetty version(s)
12

Description
Starting Jetty with JMX produces, for example:

2023-08-30 07:50:38.026:INFO :oejj.MetaData:main: Getter/setter type mismatch for mbean attribute tempDirectory in class org.eclipse.jetty.server.Server
...
2023-08-30 07:50:38.835:INFO :oejj.MetaData:main: Getter/setter type mismatch for mbean attribute configurationClasses in class org.eclipse.jetty.deploy.providers.ContextProvider

The log statement is redundant, as in both cases the matching setter exists, while overloaded versions may be there for convenience (for example when setting values from XML).

@sbordet sbordet added the Bug For general bugs on Jetty side label Aug 31, 2023
@sbordet sbordet self-assigned this Aug 31, 2023
sbordet added a commit that referenced this issue Aug 31, 2023
Now the log statement is only emitted when there is no matching setter method, but a mismatched one exists.

Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
sbordet added a commit that referenced this issue Sep 13, 2023
)

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

Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
@sbordet
Copy link
Contributor Author

sbordet commented Sep 13, 2023

Fixed by #10443.

@sbordet sbordet closed this as completed Sep 13, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Bug For general bugs on Jetty side
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant