Skip to content

[DS-6737][MasterServer] fix event handle twice#6738

Merged
lenboo merged 1 commit intoapache:devfrom
caishunfeng:fix_concurrent_event_handle
Nov 8, 2021
Merged

[DS-6737][MasterServer] fix event handle twice#6738
lenboo merged 1 commit intoapache:devfrom
caishunfeng:fix_concurrent_event_handle

Conversation

@caishunfeng
Copy link
Contributor

this pr close #6737

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.

(or)

This pull request is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 8, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 2 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

Copy link
Contributor

@lenboo lenboo left a comment

Choose a reason for hiding this comment

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

+1

@lenboo lenboo merged commit b6824b4 into apache:dev Nov 8, 2021
lenboo pushed a commit to lenboo/dolphinscheduler that referenced this pull request Nov 8, 2021
Co-authored-by: caishunfeng <534328519@qq.com>
@caishunfeng caishunfeng deleted the fix_concurrent_event_handle branch December 15, 2021 04:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] [MasterServer] Event handle twice in two thread pool

2 participants