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

Fix #4152 #4169

Merged
merged 1 commit into from
Dec 24, 2021
Merged

Fix #4152 #4169

merged 1 commit into from
Dec 24, 2021

Conversation

lonre
Copy link
Contributor

@lonre lonre commented Dec 21, 2021

beanFactory.isTypeMatch may lead to the initialization of FactoryBean,
dubbo consumer is a ReferenceBean which implements FactoryBean.
support spring 3

related to #2328 #3865 #4161 #4164

What's the purpose of this PR

XXXXX

Which issue(s) this PR fixes:

Fixes #

Brief changelog

XXXXX

Follow this checklist to help us incorporate your contribution quickly and easily:

  • Read the Contributing Guide before making this pull request.
  • Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
  • Write necessary unit tests to verify the code.
  • Run mvn clean test to make sure this pull request doesn't break anything.
  • Update the CHANGES log.

@lonre
Copy link
Contributor Author

lonre commented Dec 21, 2021

Hi @nobodyiam, I did not found nice api for checking, and it just works 😄

`beanFactory.isTypeMatch` may lead to the initialization of FactoryBean,
dubbo consumer is a ReferenceBean which implements FactoryBean.
support spring 3

related to apolloconfig#2328 apolloconfig#3865 apolloconfig#4161 apolloconfig#4164
Copy link
Member

@nobodyiam nobodyiam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nobodyiam nobodyiam merged commit e239350 into apolloconfig:1.x Dec 24, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Dec 24, 2021
@nobodyiam
Copy link
Member

@lonre Thanks for your contribution! Would you please also help to apply this patch to master branch?

@nobodyiam nobodyiam added this to the 1.9.2 milestone Jan 1, 2022
@lonre lonre deleted the fix-4512 branch April 11, 2022 14:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants