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

change alibaba_spring_context_support_version to 1.0.11 #8217

Merged

Conversation

zrlw
Copy link
Contributor

@zrlw zrlw commented Jul 4, 2021

What is the purpose of the change

  1. change alibaba spring-context-support version of dubbo pom.xml to 1.0.11
  2. set classValuesAsString and nestedAnnotationsAsMap attributes of ReferenceAnnotationBeanPostProcessor to false
    fix Dubbo 2.7.5, 2.7.9, 2.7.11, 2.7.12 are incompatiable with nacos-spring-context 1.1.0 #7797

@zrlw
Copy link
Contributor Author

zrlw commented Jul 6, 2021

@horizonzy

@codecov-commenter
Copy link

codecov-commenter commented Jul 9, 2021

Codecov Report

Merging #8217 (843448c) into master (c5a4597) will increase coverage by 0.63%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #8217      +/-   ##
============================================
+ Coverage     60.82%   61.46%   +0.63%     
- Complexity      456      491      +35     
============================================
  Files          1090     1090              
  Lines         43950    44311     +361     
  Branches       6418     6533     +115     
============================================
+ Hits          26732    27234     +502     
+ Misses        14243    14092     -151     
- Partials       2975     2985      +10     
Impacted Files Coverage Δ
...notation/ReferenceAnnotationBeanPostProcessor.java 91.93% <100.00%> (+0.13%) ⬆️
...dubbo/remoting/exchange/support/DefaultFuture.java 80.34% <0.00%> (-6.84%) ⬇️
...in/java/org/apache/dubbo/common/utils/JVMUtil.java 86.79% <0.00%> (-5.67%) ⬇️
...va/org/apache/dubbo/common/config/Environment.java 80.41% <0.00%> (-2.35%) ⬇️
...etadata/report/support/AbstractMetadataReport.java 59.70% <0.00%> (-0.98%) ⬇️
...ng/transport/dispatcher/all/AllChannelHandler.java 89.65% <0.00%> (ø)
...n/java/org/apache/dubbo/common/utils/NetUtils.java 69.30% <0.00%> (+0.33%) ⬆️
...rg/apache/dubbo/common/timer/HashedWheelTimer.java 79.72% <0.00%> (+0.34%) ⬆️
...org/apache/dubbo/registry/redis/RedisRegistry.java 58.42% <0.00%> (+0.37%) ⬆️
...he/dubbo/registry/multicast/MulticastRegistry.java 70.23% <0.00%> (+0.46%) ⬆️
... and 41 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c5a4597...843448c. Read the comment docs.

@gang0713
Copy link

这问题啥时候能解决。

@wuwen5
Copy link
Contributor

wuwen5 commented Dec 17, 2021

#7274

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.

None yet

5 participants