[Dubbo-3266] Change DynamicConfiguration definition to better adapt to Apollo's namespace storage model.

[chickenlj]

What is the purpose of the change

#3266

1. Separate the retrieving of start up configurations from that of governance rules.
2. To better adapt to Apollo's namespace model.

Brief changelog

• Add getConfigFile() api for DynamicConfiguration and the corresponding implementation in ZookeeperDynamicConfiguraiton and ApolloDynamicConfiguration
• Small optimization on ConfigCenterConfig

Verifying this change

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 integration-test in test module.
• 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 #3271 into master will increase coverage by 0.03%.
The diff coverage is 25%.

@@             Coverage Diff              @@
##             master    #3271      +/-   ##
============================================
+ Coverage     62.69%   62.73%   +0.03%     
- Complexity      544      545       +1     
============================================
  Files           762      762              
  Lines         32758    32752       -6     
  Branches       5165     5162       -3     
============================================
+ Hits          20538    20546       +8     
+ Misses         9861     9848      -13     
+ Partials       2359     2358       -1

Impacted Files	Coverage Δ	Complexity Δ
...che/dubbo/config/builders/ConfigCenterBuilder.java	100% <ø> (ø)	0 <0> (ø)	⬇️
...figcenter/support/nop/NopDynamicConfiguration.java	71.42% <0%> (-11.91%)	0 <0> (ø)
...onfigcenter/consul/ConsulDynamicConfiguration.java	0% <0%> (ø)	0 <0> (ø)	⬇️
...gcenter/support/etcd/EtcdDynamicConfiguration.java	0% <0%> (ø)	0 <0> (ø)	⬇️
...g/apache/dubbo/config/AbstractInterfaceConfig.java	72% <0%> (ø)	0 <0> (ø)	⬇️
...ter/support/apollo/ApolloDynamicConfiguration.java	0% <0%> (ø)	0 <0> (ø)	⬇️
...g/apache/dubbo/config/spring/ConfigCenterBean.java	0% <0%> (ø)	0 <0> (ø)	⬇️
...enter/support/nacos/NacosDynamicConfiguration.java	0% <0%> (ø)	0 <0> (ø)	⬇️
...pport/zookeeper/ZookeeperDynamicConfiguration.java	84% <100%> (+0.66%)	7 <1> (+1)	⬆️
...va/org/apache/dubbo/config/ConfigCenterConfig.java	76.56% <100%> (-1.05%)	0 <0> (ø)
... and 6 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 bfb6077...9244c9c. Read the comment docs.

[khanimteyaz]

@chickenlj do let me know if you planning to address the given few feedback in this PR. Otherthan given few minor comment (if looks valid to you). This PR looks good to me.

[chickenlj]

@chickenlj do let me know if you planning to address the given few feedback in this PR. Otherthan given few minor comment (if looks valid to you). This PR looks good to me.

@khanimteyaz Sure, I am working on it considering @beiwei30's and your feedbacks.

[chickenlj]

@beiwei30 @khanimteyaz Please help review the latest commits

[khanimteyaz]

LGTM.

[cvictory]

LGTM

Solved