bugfix: multi-thread concurrent add listener problem

[slievrly]

Signed-off-by: slievrly slievrly@163.com

Ⅰ. Describe what this PR did

bugfix: multi-thread concurrent add listener problem

Ⅱ. Does this pull request fix one issue?

fix #1830

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

[codecov-io]

Codecov Report

Merging #1846 into develop will decrease coverage by <.01%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #1846      +/-   ##
=============================================
- Coverage      55.06%   55.05%   -0.01%     
- Complexity      2353     2355       +2     
=============================================
  Files            422      422              
  Lines          14273    14273              
  Branches        1712     1712              
=============================================
- Hits            7859     7858       -1     
- Misses          5684     5686       +2     
+ Partials         730      729       -1

Impacted Files	Coverage Δ	Complexity Δ
...server/store/file/FileTransactionStoreManager.java	59.21% <0%> (-0.99%)	29% <0%> (ø)
...o/seata/server/coordinator/DefaultCoordinator.java	50% <0%> (+0.4%)	28% <0%> (+1%)	⬆️
...in/java/io/seata/server/session/GlobalSession.java	85.02% <0%> (+0.48%)	68% <0%> (+1%)	⬆️

[xingfudeshi]

LGTM.The other registry implementations may have similar problem.

[slievrly]

LGTM.The other registry implementations may have similar problem.

multiple listeners will not affect us, but there may be a registry that may throw a ConcurrentModificationException.

[zjinlei]

LGTM