Skip to content

typo: prvoiderService->providerService; connectSkiped->connectSkipped#2396

Merged
diecui1202 merged 1 commit intoapache:masterfrom
gongchangwangpi:master
Aug 28, 2018
Merged

typo: prvoiderService->providerService; connectSkiped->connectSkipped#2396
diecui1202 merged 1 commit intoapache:masterfrom
gongchangwangpi:master

Conversation

@gongchangwangpi
Copy link
Copy Markdown
Contributor

What is the purpose of the change

typo

Brief changelog

prvoiderService->providerService; connectSkiped->connectSkipped

Verifying this change

none

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

  • Make sure there is a GITHUB_issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
  • Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
  • Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
  • Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
  • Run mvn clean install -DskipTests & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
  • If this contribution is large, please follow the Software Donation Guide.

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #2396 into master will increase coverage by 0.04%.
The diff coverage is 33.33%.

Impacted file tree graph

@@             Coverage Diff             @@
##             master   #2396      +/-   ##
===========================================
+ Coverage     54.76%   54.8%   +0.04%     
- Complexity     5234    5237       +3     
===========================================
  Files           573     573              
  Lines         25453   25456       +3     
  Branches       4537    4537              
===========================================
+ Hits          13939   13951      +12     
+ Misses         9416    9410       -6     
+ Partials       2098    2095       -3
Impacted Files Coverage Δ Complexity Δ
...org/apache/dubbo/registry/redis/RedisRegistry.java 32.76% <33.33%> (ø) 16 <0> (ø) ⬇️
...he/dubbo/remoting/transport/netty/NettyServer.java 67.85% <0%> (-3.58%) 8% <0%> (-1%)
.../apache/dubbo/rpc/protocol/dubbo/DubboInvoker.java 68.33% <0%> (-3.34%) 11% <0%> (ø)
...ache/dubbo/remoting/transport/mina/MinaClient.java 56.92% <0%> (-2.46%) 8% <0%> (-1%)
...bo/remoting/transport/netty/NettyCodecAdapter.java 53.12% <0%> (-1.57%) 3% <0%> (ø)
...he/dubbo/remoting/transport/netty/NettyClient.java 72.88% <0%> (-1.26%) 12% <0%> (ø)
...e/dubbo/remoting/transport/netty4/NettyClient.java 63.33% <0%> (-1.08%) 11% <0%> (ø)
...pache/dubbo/registry/support/AbstractRegistry.java 69.28% <0%> (+0.74%) 63% <0%> (+1%) ⬆️
...a/org/apache/dubbo/monitor/dubbo/DubboMonitor.java 89.71% <0%> (+1.86%) 15% <0%> (ø) ⬇️
...e/dubbo/remoting/transport/netty4/NettyServer.java 75.4% <0%> (+3.27%) 10% <0%> (+1%) ⬆️
... and 2 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 5f8adf0...95bf819. Read the comment docs.

@diecui1202
Copy link
Copy Markdown

@363230482 Thanks for u contribution. It looks good to me.

@diecui1202 diecui1202 merged commit 86d7347 into apache:master Aug 28, 2018
zzzzhougit pushed a commit to zzzzhougit/incubator-dubbo that referenced this pull request Sep 9, 2018
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