Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature-14321][API] Support to complement data in descending or ascending order of date #14450

Merged
merged 10 commits into from
Jul 5, 2023

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

songjianet
songjianet previously approved these changes Jul 5, 2023
@sonarcloud
Copy link

sonarcloud 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
48 checks passed
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
biaoma-ty pushed a commit to Kasma-Inc/dolphinscheduler that referenced this pull request Aug 17, 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