Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Revisit public API for MBeanExporter #30874

Closed
sbrannen opened this issue Jul 12, 2023 · 1 comment
Closed

Revisit public API for MBeanExporter #30874

sbrannen opened this issue Jul 12, 2023 · 1 comment
Assignees
@jhoeller
Labels
in: core Issues in core modules (aop, beans, core, context, expression) type: enhancement A general enhancement
Milestone
6.1.0-M3

Comments

@sbrannen
Copy link
Member

sbrannen commented Jul 12, 2023 •
edited
Loading

Overview

Inspired by #30846, we should revisit the public API for MBeanExporter, potentially deprecating the AUTODETECT_ constants, etc.

If we deprecate those constants and the corresponding setter methods, we should close #17728.

Related Issues
@sbrannen sbrannen added in: core Issues in core modules (aop, beans, core, context, expression) type: enhancement A general enhancement labels Jul 12, 2023
@sbrannen sbrannen modified the milestones: 6.1.x, 6.1.0-M3 Jul 12, 2023
@jhoeller jhoeller self-assigned this Jul 12, 2023
@jhoeller jhoeller mentioned this issue Jul 13, 2023
Closed
@jhoeller
Copy link
Contributor

With AnnotationMBeanExporter in the picture, we can definitely deprecate the autowire constants. From an initial pass, nothing else jumps out as equally outdated, so from a backwards compatibility perspective it's probably better to preserve the remaining configuration options as-is.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jhoeller jhoeller

Labels
in: core Issues in core modules (aop, beans, core, context, expression) type: enhancement A general enhancement
Projects
None yet
Milestone
6.1.0-M3
Development

No branches or pull requests
2 participants
@sbrannen @jhoeller