Skip to content

[Feature] 2.0.2-prepare bug fix of Pressure tests #7511#7540

Merged
lenboo merged 14 commits intoapache:2.0.2-preparefrom
zwZjut:#7511
Dec 22, 2021
Merged

[Feature] 2.0.2-prepare bug fix of Pressure tests #7511#7540
lenboo merged 14 commits intoapache:2.0.2-preparefrom
zwZjut:#7511

Conversation

@zwZjut
Copy link
Copy Markdown
Contributor

@zwZjut zwZjut commented Dec 22, 2021 •
edited
Loading

Purpose of the pull request

#7511

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 11 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
to #7511
3b24c4b
@zwZjut
Copy link
Copy Markdown
Contributor Author

zwZjut commented Dec 22, 2021

#7511

@zwZjut zwZjut changed the base branch from dev to 2.0.2-prepare December 22, 2021 04:28
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Dec 22, 2021 •
edited
Loading

Codecov Report

❌ Patch coverage is 14.35644% with 173 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (2.0.2-prepare@0b6f3fb). Learn more about missing BASE report.

Files with missing lines Patch % Lines
...r/server/master/registry/MasterRegistryClient.java 27.77% 43 Missing and 9 partials ⚠️
...r/server/worker/registry/WorkerRegistryClient.java 0.00% 26 Missing ⚠️
...er/server/master/runner/FailoverExecuteThread.java 0.00% 22 Missing ⚠️
...er/master/processor/queue/TaskResponseService.java 27.77% 13 Missing ⚠️
...er/server/master/runner/WorkflowExecuteThread.java 0.00% 9 Missing ⚠️
...e/dolphinscheduler/server/master/MasterServer.java 0.00% 8 Missing ⚠️
...e/dolphinscheduler/server/worker/WorkerServer.java 0.00% 8 Missing ⚠️
...lphinscheduler/service/process/ProcessService.java 30.00% 7 Missing ⚠️
...inscheduler/server/master/config/MasterConfig.java 0.00% 6 Missing ⚠️
...ver/master/consumer/TaskPriorityQueueConsumer.java 0.00% 5 Missing ⚠️
... and 11 more
Additional details and impacted files 
@@               Coverage Diff                @@
##             2.0.2-prepare    #7540   +/-   ##
================================================
  Coverage                 ?   31.72%           
  Complexity               ?     1572           
================================================
  Files                    ?      433           
  Lines                    ?    14716           
  Branches                 ?     1465           
================================================
  Hits                     ?     4669           
  Misses                   ?     9589           
  Partials                 ?      458

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

caishunfeng
caishunfeng reviewed Dec 22, 2021
View reviewed changes
Comment thread ...inscheduler-server/src/main/java/org/apache/dolphinscheduler/server/worker/WorkerServer.java
Comment thread ...r/src/main/java/org/apache/dolphinscheduler/server/master/registry/MasterRegistryClient.java
Comment thread ...r/src/main/java/org/apache/dolphinscheduler/server/master/registry/MasterRegistryClient.java
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Dec 22, 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 317 Code Smells

17.3% 17.3% Coverage
1.1% 1.1% Duplication

@caishunfeng caishunfeng added this to the 2.0.2-release milestone Dec 22, 2021
lenboo
lenboo approved these changes Dec 22, 2021
View reviewed changes
Copy link
Copy Markdown
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 c956185 into apache:2.0.2-prepare Dec 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@caishunfeng caishunfeng caishunfeng left review comments

@lenboo lenboo lenboo approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

2.0.2-release

Development

Successfully merging this pull request may close these issues.

4 participants

@zwZjut @codecov-commenter @caishunfeng @lenboo