Skip to content

[hotfix] Clear thread name in BaseTaskProcessor#11422

Merged
caishunfeng merged 1 commit intoapache:devfrom
ruanwenjun:dev_wenjun_clearThreadNameInBaseTaskProcessor
Aug 12, 2022
Merged

[hotfix] Clear thread name in BaseTaskProcessor#11422
caishunfeng merged 1 commit intoapache:devfrom
ruanwenjun:dev_wenjun_clearThreadNameInBaseTaskProcessor

Conversation

@ruanwenjun
Copy link
Copy Markdown
Member

Purpose of the pull request

Clear the thread name to avoid the log print bug

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:

(or)

If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #11422 (7129b35) into dev (496c2d4) will increase coverage by 0.00%.
The diff coverage is 47.36%.

@@            Coverage Diff            @@
##                dev   #11422   +/-   ##
=========================================
  Coverage     39.43%   39.44%           
+ Complexity     4622     4621    -1     
=========================================
  Files           980      980           
  Lines         37260    37268    +8     
  Branches       4176     4176           
=========================================
+ Hits          14695    14700    +5     
- Misses        21031    21033    +2     
- Partials       1534     1535    +1     
Impacted Files Coverage Δ
...r/server/master/runner/task/BaseTaskProcessor.java 15.12% <47.36%> (+1.82%) ⬆️
...er/master/dispatch/host/assign/RandomSelector.java 77.77% <0.00%> (-5.56%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@sonarqubecloud
Copy link
Copy Markdown

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 0 Code Smells

50.0% 50.0% Coverage
0.0% 0.0% Duplication

Copy link
Copy Markdown
Contributor

@caishunfeng caishunfeng left a comment

Choose a reason for hiding this comment

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

👍

@caishunfeng caishunfeng merged commit 47b5006 into apache:dev Aug 12, 2022
@caishunfeng caishunfeng added this to the 3.0.1 milestone Aug 12, 2022
@ruanwenjun ruanwenjun deleted the dev_wenjun_clearThreadNameInBaseTaskProcessor branch August 12, 2022 02:16
ruanwenjun added a commit to ruanwenjun/dolphinscheduler that referenced this pull request Aug 12, 2022
ruanwenjun added a commit to ruanwenjun/dolphinscheduler that referenced this pull request Aug 13, 2022
ruanwenjun added a commit to ruanwenjun/dolphinscheduler that referenced this pull request Sep 17, 2022
@caishunfeng caishunfeng added the release cherry-pick Mark this issue/PR had cherry-pick for release version label Sep 17, 2022
caishunfeng pushed a commit that referenced this pull request Sep 17, 2022
xdu-chenrj pushed a commit to xdu-chenrj/dolphinscheduler that referenced this pull request Oct 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend release cherry-pick Mark this issue/PR had cherry-pick for release version

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants