Make changes according to the suggestions of the review

[chenpiaoping]

No description provided.

[xieus]

LGTM. Sorry to merge #515 too early :-)

[codecov-io]

Codecov Report

Merging #519 (780b9ac) into master (28cfda5) will decrease coverage by 0.57%.
The diff coverage is 44.44%.

@@             Coverage Diff              @@
##             master     #519      +/-   ##
============================================
- Coverage     35.72%   35.14%   -0.58%     
+ Complexity     1163     1150      -13     
============================================
  Files           468      469       +1     
  Lines         11318    11384      +66     
  Branches       1456     1464       +8     
============================================
- Hits           4043     4001      -42     
- Misses         6723     6842     +119     
+ Partials        552      541      -11     

Impacted Files	Coverage Δ	Complexity Δ
...alcor/portmanager/processor/FixedIpsProcessor.java	69.73% <ø> (-0.14%)	41.00 <0.00> (ø)
...ewei/alcor/portmanager/processor/MacProcessor.java	91.93% <ø> (-1.72%)	12.00 <0.00> (-1.00)
.../portmanager/processor/SecurityGroupProcessor.java	81.42% <ø> (-0.27%)	17.00 <0.00> (ø)
...alcor/portmanager/proxy/DataPlaneManagerProxy.java	0.00% <ø> (ø)	0.00 <0.00> (ø)
...turewei/alcor/portmanager/repo/PortRepository.java	45.00% <0.00%> (-20.57%)	28.00 <0.00> (ø)
...ortmanager/request/CreateNetworkConfigRequest.java	70.00% <ø> (ø)	3.00 <0.00> (ø)
...ortmanager/request/DeleteNetworkConfigRequest.java	70.00% <ø> (ø)	3.00 <0.00> (ø)
...ortmanager/request/UpdateNetworkConfigRequest.java	77.77% <ø> (ø)	3.00 <0.00> (ø)
...tmanager/rollback/CreateNetworkConfigRollback.java	0.00% <ø> (ø)	0.00 <0.00> (ø)
...tmanager/rollback/DeleteNetworkConfigRollback.java	0.00% <ø> (ø)	0.00 <0.00> (ø)
... and 27 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 488e422...780b9ac. Read the comment docs.