bugfix: use Set to avoid duplicate listeners. fixes #1126

[kezhenxu94]

Ⅰ. Describe what this PR did

This patch replaces ArrayList with HaseSet to avoid duplicated listeners as described in #1126 , and also fix some typos

Ⅱ. Does this pull request fix one issue?

fixes #1126

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

Already have

Ⅳ. Describe how to verify it

CI passes

Ⅴ. Special notes for reviews

null

[codecov-io]

Codecov Report

Merging #1169 into develop will decrease coverage by 0.02%.
The diff coverage is 83.33%.

@@             Coverage Diff              @@
##             develop   #1169      +/-   ##
============================================
- Coverage      43.42%   43.4%   -0.03%     
  Complexity      1428    1428              
============================================
  Files            250     250              
  Lines          10224   10222       -2     
  Branches        1333    1333              
============================================
- Hits            4440    4437       -3     
  Misses          5202    5202              
- Partials         582     583       +1

Impacted Files	Coverage Δ	Complexity Δ
...in/java/io/seata/server/session/GlobalSession.java	79.8% <83.33%> (-0.2%)	59 <3> (ø)
...server/store/file/FileTransactionStoreManager.java	45.99% <0%> (-0.35%)	19% <0%> (ø)

---

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 9ee5c32...ec73946. Read the comment docs.

[leizhiyuan]

See the comment

[slievrly]

See the comment

Is the hashset's order of addition and traversal order consistent? There is currently no order requirement in the business.

[leizhiyuan]

See the comment

Is the hashset's order of addition and traversal order consistent? There is currently no order requirement in the business.

yes，what I mean is the order of hashset, but the method name is 'getSortedBranches' maybe I think that sort it well..

[slievrly]

See the comment

Is the hashset's order of addition and traversal order consistent? There is currently no order requirement in the business.

yes，what I mean is the order of hashset, but the method name is 'getSortedBranches' maybe I think that sort it well..

yeah, getSortedBranches need sort, but it's a ArrayList used Array copy method.

[leizhiyuan]

LGTM