Skip to content

Conversation

@calvinjiang
Copy link
Contributor

Purpose of the pull request

This PR will close #14321 .

Brief change log

Verify this pull request

This change added tests and can be verified as follows:

  • Manually verified the change by testing locally.

image

image

@calvinjiang calvinjiang added feature new feature backend 3.2.0 for 3.2.0 version labels Jul 4, 2023
@calvinjiang calvinjiang added this to the 3.2.0 milestone Jul 4, 2023
@github-actions github-actions bot added the UI ui and front end related label Jul 4, 2023
@codecov-commenter
Copy link

codecov-commenter commented Jul 4, 2023

Codecov Report

Merging #14450 (6926458) into dev (9cbb262) will decrease coverage by 0.01%.
The diff coverage is 60.86%.

❗ Current head 6926458 differs from pull request most recent head 6d4e720. Consider uploading reports for the commit 6d4e720 to get more accurate results

@@             Coverage Diff              @@
##                dev   #14450      +/-   ##
============================================
- Coverage     38.52%   38.52%   -0.01%     
  Complexity     4555     4555              
============================================
  Files          1236     1237       +1     
  Lines         43486    43462      -24     
  Branches       4817     4814       -3     
============================================
- Hits          16755    16743      -12     
+ Misses        24875    24861      -14     
- Partials       1856     1858       +2     
Impacted Files Coverage Δ
...inscheduler/api/controller/ExecutorController.java 17.94% <ø> (ø)
.../dolphinscheduler/common/enums/ExecutionOrder.java 0.00% <0.00%> (ø)
.../plugin/task/api/parameters/K8sTaskParameters.java 0.00% <ø> (ø)
...i/service/impl/AlertPluginInstanceServiceImpl.java 63.86% <33.33%> (ø)
...cheduler/api/service/impl/ExecutorServiceImpl.java 53.13% <61.90%> (+1.72%) ⬆️
...uler/plugin/task/api/k8s/impl/K8sTaskExecutor.java 38.29% <92.30%> (-0.21%) ⬇️
.../org/apache/dolphinscheduler/api/enums/Status.java 100.00% <100.00%> (ø)
...che/dolphinscheduler/api/python/PythonGateway.java 17.29% <100.00%> (+0.35%) ⬆️
...ache/dolphinscheduler/plugin/task/k8s/K8sTask.java 80.95% <100.00%> (+0.46%) ⬆️

... and 3 files with indirect coverage changes

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

@calvinjiang calvinjiang requested a review from zhuangchong July 4, 2023 12:30
songjianet
songjianet previously approved these changes Jul 5, 2023
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jul 5, 2023

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

69.0% 69.0% Coverage
0.0% 0.0% Duplication

@zhuangchong zhuangchong merged commit 76b1eef into apache:dev Jul 5, 2023
IT-Kwj pushed a commit to IT-Kwj/dolphinscheduler that referenced this pull request Jul 14, 2023
zhongjiajie pushed a commit that referenced this pull request Jul 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3.2.0 for 3.2.0 version backend feature new feature UI ui and front end related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature][API] Support to complement data in descending or ascending order of date.

4 participants