Skip to content

[Bug] [dolphinscheduler-server] workflow is always running when worker sendResult success but master not received #7610

Merged
caishunfeng merged 14 commits intoapache:2.0.2-preparefrom
zwZjut:#7609
Dec 24, 2021
Merged

[Bug] [dolphinscheduler-server] workflow is always running when worker sendResult success but master not received #7610
caishunfeng merged 14 commits intoapache:2.0.2-preparefrom
zwZjut:#7609

Conversation

@zwZjut
Copy link
Copy Markdown
Contributor

@zwZjut zwZjut commented Dec 24, 2021

Purpose of the pull request

#7609

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:

zwZjut and others added 14 commits December 13, 2021 10:24
@zwZjut
[Feature][dolphinscheduler-api] parse traceId in http header for Cros…
2f26f5f 
…s system delivery to #7237 (#7238)

* to #7237

* rerun test

Co-authored-by: honghuo.zw <honghuo.zw@alibaba-inc.com>
@zwZjut
chery-pick 05aef27 and handle conflicts
4a90027
@zwZjut
to #7065: fix ExecutorService and schedulerService (#7072)
9c45a4f 
Co-authored-by: honghuo.zw <honghuo.zw@alibaba-inc.com>
@zwZjut
[Feature][dolphinscheduler-api] access control of taskDefinition and …
6897517 
…taskInstance in project to #7081  (#7082)

* to #7081

* fix #7081

* to #7081

Co-authored-by: honghuo.zw <honghuo.zw@alibaba-inc.com>
@zwZjut
chery-pick 8ebe060 and handle conflicts
6f491e4
@zwZjut
cherry-pick 1f18444 and handle conflicts
1bcde64
@zwZjut @CalvinKirs
fix #6807: dolphinscheduler.zookeeper.env_vars - > dolphinscheduler.r…
9cddad9 
…egistry.env_vars (#6808)

Co-authored-by: honghuo.zw <honghuo.zw@alibaba-inc.com>
Co-authored-by: Kirs <acm_master@163.com>
@zwZjut
add default constructor (#6780)
70ec96a 
Co-authored-by: honghuo.zw <honghuo.zw@alibaba-inc.com>
@zwZjut
to #7108 (#7109)
c79acd1
Merge branch '2.0.2-prepare' of https://github.com/zwZjut/dolphinsche…
6c29e89 
…duler into 2.0.2-prepare
Merge branch '2.0.2-prepare' of https://github.com/zwZjut/dolphinsche…
3367f56 
…duler into 2.0.2-prepare
Merge branch '2.0.2-prepare' of https://github.com/zwZjut/dolphinsche…
9737335 
…duler into 2.0.2-prepare
Merge branch '2.0.2-prepare' of https://github.com/zwZjut/dolphinsche…
8842fe6 
…duler into 2.0.2-prepare
to #7609
2399f68
@zwZjut zwZjut changed the title #7609 [Bug] [dolphinscheduler-server] workflow is always running when worker sendResult success but master not received Dec 24, 2021
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Dec 24, 2021

Codecov Report

❗ No coverage uploaded for pull request base (2.0.2-prepare@f8942bf). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff                @@
##             2.0.2-prepare    #7610   +/-   ##
================================================
  Coverage                 ?   31.43%           
  Complexity               ?     1559           
================================================
  Files                    ?      434           
  Lines                    ?    14813           
  Branches                 ?     1478           
================================================
  Hits                     ?     4656           
  Misses                   ?     9702           
  Partials                 ?      455

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 f8942bf...2399f68. Read the comment docs.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Dec 24, 2021

SonarCloud Quality Gate failed.    Quality Gate failed

Bug E 24 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot E 3 Security Hotspots
Code Smell A 321 Code Smells

16.8% 16.8% Coverage
0.9% 0.9% Duplication

caishunfeng
caishunfeng approved these changes Dec 24, 2021
View reviewed changes
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.

+1

@caishunfeng caishunfeng merged commit 6c819ee into apache:2.0.2-prepare Dec 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@caishunfeng caishunfeng caishunfeng approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@zwZjut @codecov-commenter @caishunfeng