issues87 修复在多个/多级SpringContext并存环境下,EventPublishingConfigService发送事件混乱,引发后续依赖事… #260
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
…件机制的系列问题,如@NacosValue注解无法解析到最新值
Please do not create a Pull Request without creating an issue first.
What is the purpose of the change
修复在多个/多级SpringContext并存环境下,EventPublishingConfigService发送事件混乱,引发后续依赖事件机制的系列问题,如@NacosValue注解无法解析到最新值
Brief changelog
Verifying this change
2.CacheableEventPublishingNacosServiceFactory拥有字段(ConfigurableApplicationContext context),它能作为单例使用的前提条件是,应用的只有存在一个容器单独使用Nacos,而多个SpringContext同时使用Nacos,就可能引发创建的ConfigService是(上一个SpringContext创建的,被缓存)
3.基于以上,此PR删除CacheableEventPublishingNacosServiceFactory.getSingleton()获取单例的方法,将CacheableEventPublishingNacosServiceFactory对象的创建关联到容器
Follow this checklist to help us incorporate your contribution quickly and easily:
[ISSUE #123] Fix UnknownException when host config not exist
. Each commit in the pull request should have a meaningful subject line and body.mvn -B clean package apache-rat:check findbugs:findbugs -Dmaven.test.skip=true
to make sure basic checks pass. Runmvn clean install -DskipITs
to make sure unit-test pass. Runmvn clean test-compile failsafe:integration-test
to make sure integration-test pass.