fix for #4175: ServiceConfigurationListener should override ProviderConfigurationListener

[uglycow]

What is the purpose of the change

fix for #4175: ServiceConfigurationListener should override ProviderConfigurationListener

Brief changelog

XXXXX

Verifying this change

XXXXX

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

• Make sure there is a GITHUB_issue field 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 sample in dubbo samples project.
• Run mvn clean install -DskipTests=false & 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]

Codecov Report

Merging #4179 into master will decrease coverage by 0.06%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #4179      +/-   ##
============================================
- Coverage     62.91%   62.85%   -0.07%     
+ Complexity      508      506       -2     
============================================
  Files           768      768              
  Lines         33059    33059              
  Branches       5218     5218              
============================================
- Hits          20800    20780      -20     
- Misses         9867     9884      +17     
- Partials       2392     2395       +3

Impacted Files	Coverage Δ	Complexity Δ
...e/dubbo/registry/integration/RegistryProtocol.java	82.69% <100%> (ø)	0 <0> (ø)	⬇️
...ache/dubbo/remoting/p2p/support/AbstractGroup.java	45.45% <0%> (-11.37%)	0% <0%> (ø)
.../apache/dubbo/remoting/transport/AbstractPeer.java	60.86% <0%> (-10.87%)	0% <0%> (ø)
...apache/dubbo/common/config/ConfigurationUtils.java	64% <0%> (-8%)	0% <0%> (ø)
...mmon/threadpool/support/AbortPolicyWithReport.java	83.78% <0%> (-5.41%)	0% <0%> (ø)
...onfig/spring/extension/SpringExtensionFactory.java	80.48% <0%> (-4.88%)	0% <0%> (ø)
...bo/rpc/cluster/support/FailbackClusterInvoker.java	67.21% <0%> (-3.28%)	0% <0%> (ø)
...pache/dubbo/registry/support/AbstractRegistry.java	78.54% <0%> (-3.07%)	0% <0%> (ø)
...he/dubbo/registry/multicast/MulticastRegistry.java	67.87% <0%> (-1.36%)	0% <0%> (ø)
...che/dubbo/remoting/transport/mina/MinaChannel.java	53.94% <0%> (-1.32%)	17% <0%> (-1%)
... and 8 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 3dd261a...a040575. Read the comment docs.

[Leishunyu]

I think it would be better to add this comment, service > application

[ralf0131]

LGTM.