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

refactor:optimize the order and condition matching of service registration automatic configuration. #1132

Conversation

hligaty
Copy link
Contributor

@hligaty hligaty commented Sep 22, 2023

PR Type

Refactoring (no functional changes, no api changes).

Describe what this PR does for and how you did.

服务注册自动配置有下面几个疑问:

  1. @AutoConfigureAfter 中 AutoServiceRegistrationConfiguration.class 应该是无效的。解决:移除这个参数
  2. PolarisServiceRegistry 定义需要在 ServiceRegistryAutoConfiguration 完成前执行。解决:添加 @AutoConfigureBefore
  3. 判断是否进行注册服务的条件注解放在服务注册自动配置类上覆盖所有 bean 会更清晰?

Adding the issue link (#xxx) if possible.

Note

Checklist

  • Add information of this PR to CHANGELOG.md in root of project.
  • Add documentation in javadoc or comment below the PR if necessary.

Checklist (Optional)

  • Will pull request to branch of 2020.0.
  • Will pull request to branch of 2022.0.

SkyeBeFreeman
SkyeBeFreeman previously approved these changes Sep 22, 2023
…tching_of_service_registration_automatic_configuration
@SkyeBeFreeman SkyeBeFreeman merged commit fa75fad into Tencent:2020.0 Sep 22, 2023
8 checks passed
@hligaty hligaty deleted the 2020.0-optimize_the_order_and_condition_matching_of_service_registration_automatic_configuration branch September 22, 2023 07:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants