Skip to content

[cherry-pick-2.0.2] #7530 #7506#7570

Merged
caishunfeng merged 2 commits intoapache:2.0.2-preparefrom
caishunfeng:cherry-pick-2.0.2
Dec 23, 2021
Merged

[cherry-pick-2.0.2] #7530 #7506#7570
caishunfeng merged 2 commits intoapache:2.0.2-preparefrom
caishunfeng:cherry-pick-2.0.2

Conversation

@caishunfeng
Copy link
Copy Markdown
Contributor

@caishunfeng caishunfeng commented Dec 23, 2021

cherry-pick #7530 #7506
issue : #7529

Purpose of the pull request

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:

Copy link
Copy Markdown
Member

@kezhenxu94 kezhenxu94 left a comment

Choose a reason for hiding this comment

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

👍

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Dec 23, 2021

Codecov Report

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

❗ Current head a00c4f9 differs from pull request most recent head 658fce0. Consider uploading reports for the commit 658fce0 to get more accurate results
Impacted file tree graph

@@               Coverage Diff                @@
##             2.0.2-prepare    #7570   +/-   ##
================================================
  Coverage                 ?   41.40%           
  Complexity               ?     3682           
================================================
  Files                    ?      641           
  Lines                    ?    26473           
  Branches                 ?     2961           
================================================
  Hits                     ?    10961           
  Misses                   ?    14481           
  Partials                 ?     1031           

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 281cea8...658fce0. Read the comment docs.

datasource:
driver-class-name: org.h2.Driver
url: jdbc:h2:mem:dolphinscheduler;MODE=MySQL;DB_CLOSE_DELAY=-1;DATABASE_TO_LOWER=true;INIT=runscript from 'classpath:sql/dolphinscheduler_h2.sql'
url: jdbc:h2:mem:dolphinscheduler;MODE=MySQL;DB_CLOSE_DELAY=-1;DATABASE_TO_LOWER=true
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

You missed this part

sql:
init:
schema-locations: classpath:sql/dolphinscheduler_h2.sql

lgcareer and others added 2 commits December 23, 2021 15:04
…#7506)

[DS-7227][refactor] optimize alert plugin instance list order
query all alert plugin instance list order by update time desc
optimize `select *` sql in the AlterPluginInstanceMapper.xml
@caishunfeng caishunfeng merged commit 1693405 into apache:2.0.2-prepare Dec 23, 2021
@caishunfeng caishunfeng changed the title [cherry-pick-2.0.2] #7556 #7530 #7506 [cherry-pick-2.0.2] #7530 #7506 Dec 23, 2021
@sonarqubecloud
Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants