[Feature-2815][server] One worker can belong to different workergroups

[yangyichao-mango]

What is the purpose of the pull request

Implement one worker can belong to different workergroups(#2591 ) (#2815 )

Brief change log

• Change worker.properties file's worker.group conf to worker.groups
• Change server/worker/config/WorkerConfig#workerGroup to server/worker/config/WorkerConfig#workerGroups
• Change workerRegistry to registry one worker to muti-groups
• Add tests for this issue

Verify this pull request

This change added tests and can be verified as follows:

• Changed WorkerRegistryTest to verify the change.

[codecov-commenter]

Codecov Report

Merging #2934 into dev will decrease coverage by 0.13%.
The diff coverage is 1.09%.

@@             Coverage Diff              @@
##                dev    #2934      +/-   ##
============================================
- Coverage     33.52%   33.38%   -0.14%     
+ Complexity     2393     2389       -4     
============================================
  Files           442      443       +1     
  Lines         20596    20648      +52     
  Branches       2522     2526       +4     
============================================
- Hits           6905     6894      -11     
- Misses        13029    13094      +65     
+ Partials        662      660       -2     

Impacted Files	Coverage Δ	Complexity Δ
...lphinscheduler/common/model/WorkerServerModel.java	0.00% <0.00%> (ø)	0.00 <0.00> (?)
...heduler/server/master/registry/MasterRegistry.java	0.00% <0.00%> (ø)	0.00 <0.00> (ø)
...olphinscheduler/server/registry/HeartBeatTask.java	0.00% <0.00%> (ø)	0.00 <0.00> (ø)
...inscheduler/server/worker/config/WorkerConfig.java	0.00% <0.00%> (ø)	0.00 <0.00> (ø)
...heduler/server/worker/registry/WorkerRegistry.java	0.00% <0.00%> (ø)	0.00 <0.00> (ø)
...e/dolphinscheduler/api/service/MonitorService.java	13.63% <4.34%> (-11.37%)	2.00 <1.00> (ø)
...he/dolphinscheduler/common/enums/SqoopJobType.java	0.00% <0.00%> (-88.89%)	0.00% <0.00%> (-3.00%)
...he/dolphinscheduler/api/service/LoggerService.java	82.60% <0.00%> (-8.70%)	9.00% <0.00%> (-1.00%)
...inscheduler/common/task/sqoop/SqoopParameters.java	74.00% <0.00%> (-2.00%)	25.00% <0.00%> (ø%)

---

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 d1282fd...db9de61. Read the comment docs.

[davidzollo]

sorry for not review in time, please solve conflicting files first

[yangyichao-mango]

Thx a lot for your review @dailidong , I've rebased the branch and resolved the conflicts.

[davidzollo]

+1

good job

[qiaozhanwei]

+1 please resove conflicting , Thx

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities (and 0 Security Hotspots to review)
1 Code Smell

41.6% Coverage
0.0% Duplication

[yangyichao-mango]

+1 please resove conflicting , Thx

Hi,
Thx a lot for your review, I have resolved the conflicts.