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

[master] fix #9847 #9848

Merged
merged 1 commit into from
Apr 1, 2022
Merged

[master] fix #9847 #9848

merged 1 commit into from
Apr 1, 2022

Conversation

wangchengming666
Copy link
Member

@wangchengming666 wangchengming666 commented Mar 29, 2022

fix #9847

翻了以前的一些issue 比如 #8212#5122 都提到过这个问题,我认为这段强校验的代码没必要存在。

    if (actualTypeArgsLength != 2) {
        throw new IllegalArgumentException(MessageFormat.format(
                "[ServiceDefinitionBuilder] Map type [{0}] with unexpected amount of arguments [{1}]."
                        + Arrays.toString(actualTypeArgs), type, actualTypeArgs));
    }

虽然可以通过SPI来扩展,但是还是给用户带来了一定的困扰。

@codecov-commenter
Copy link

codecov-commenter commented Mar 29, 2022

Codecov Report

Merging #9848 (f29433b) into master (1222676) will increase coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #9848      +/-   ##
============================================
+ Coverage     60.87%   60.89%   +0.02%     
+ Complexity      448      447       -1     
============================================
  Files          1100     1100              
  Lines         44560    44557       -3     
  Branches       6487     6486       -1     
============================================
+ Hits          27125    27135      +10     
+ Misses        14464    14456       -8     
+ Partials       2971     2966       -5     
Impacted Files Coverage Δ
...bo/metadata/definition/builder/MapTypeBuilder.java 71.42% <ø> (+8.92%) ⬆️
...he/dubbo/remoting/transport/netty/NettyServer.java 70.17% <0.00%> (-3.51%) ⬇️
.../exchange/support/header/HeaderExchangeServer.java 67.61% <0.00%> (-1.91%) ⬇️
...ng/zookeeper/curator5/Curator5ZookeeperClient.java 51.97% <0.00%> (-1.13%) ⬇️
.../apache/dubbo/config/bootstrap/DubboBootstrap.java 54.62% <0.00%> (+0.15%) ⬆️
...apache/dubbo/common/extension/ExtensionLoader.java 80.41% <0.00%> (+0.41%) ⬆️
...etadata/report/support/AbstractMetadataReport.java 59.70% <0.00%> (+3.39%) ⬆️
.../apache/dubbo/remoting/transport/AbstractPeer.java 63.04% <0.00%> (+4.34%) ⬆️
...ng/transport/dispatcher/all/AllChannelHandler.java 89.65% <0.00%> (+6.89%) ⬆️
...port/identifier/BaseServiceMetadataIdentifier.java 60.71% <0.00%> (+7.14%) ⬆️

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 1222676...f29433b. Read the comment docs.

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.

3 participants