Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

将 listeners 更改为 CopyOnWriteArraySet #2519

Merged
merged 1 commit into from
Jan 1, 2022
Merged

将 listeners 更改为 CopyOnWriteArraySet #2519

merged 1 commit into from
Jan 1, 2022

Conversation

howiekang
Copy link
Contributor

Describe what this PR does / why we need it

将 listeners 更改为 CopyOnWriteArraySet,以应对并发修改listeners 会存在集合快速失败的问题

Does this pull request fix one issue?

#2497

Describe how you did it

Fixes #2497

@CLAassistant
Copy link

CLAassistant commented Dec 31, 2021

CLA assistant check
All committers have signed the CLA.

@sczyh30 sczyh30 added the kind/enhancement Category issues or prs related to enhancement. label Jan 1, 2022
Copy link
Member

@sczyh30 sczyh30 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sczyh30 sczyh30 merged commit 02cfe40 into alibaba:master Jan 1, 2022
@sczyh30
Copy link
Member

sczyh30 commented Jan 1, 2022

Thanks for contributing!

Zhang-0952 pushed a commit to Zhang-0952/Sentinel that referenced this pull request Mar 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement Category issues or prs related to enhancement.
Projects
None yet
3 participants