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.

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

Deprecate ReactorNettySender #2814

Merged

Conversation

shakuzen
Copy link
Member

Having code in micrometer-core that depends on reactor-netty creates a dependency cycle between the two projects. In an effort to work towards eliminating that cycle, we are proactively deprecating the ReactorNettySender. If we receive feedback that users have use cases for it, we can consider moving it to the reactor-netty project or elsewhere. Lacking enough feedback, we will plan to remove this in the next feature release.

See gh-2802

Having code in micrometer-core that depends on reactor-netty creates a dependency cycle between the two projects. In an effort to work towards eliminating that cycle, we are proactively deprecating the ReactorNettySender. If we receive feedback that users have use cases for it, we can consider moving it to the reactor-netty project or elsewhere. Lacking enough feedback, we will plan to remove this in the next feature release.

See micrometer-metricsgh-2802
@shakuzen shakuzen added type: task A general task release notes Noteworthy change to call out in the release notes module: micrometer-core An issue that is related to our core module labels Oct 13, 2021
@shakuzen shakuzen added this to the 1.8.0-RC1 milestone Oct 13, 2021
@shakuzen shakuzen changed the title Deprecate ReactorNettySender Deprecate ReactorNettySender Oct 13, 2021
@jonatan-ivanov jonatan-ivanov merged commit fc2bae0 into micrometer-metrics:main Oct 13, 2021
@shakuzen shakuzen deleted the dr-reactor-netty-deprecator branch October 14, 2021 01:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module: micrometer-core An issue that is related to our core module release notes Noteworthy change to call out in the release notes type: task A general task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants