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

Update dolphinscheduler_ddl.sql - for PostgreSQL #13081

Merged
merged 1 commit into from
Dec 3, 2022

Conversation

darrkz
Copy link
Contributor

@darrkz darrkz commented Dec 2, 2022

#12826 for this pr

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:

(or)

If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

@DarkAssassinator
Copy link
Contributor

hi @darkz1984 , same as #13080 ?

@codecov-commenter
Copy link

codecov-commenter commented Dec 2, 2022

Codecov Report

Merging #13081 (a9dd40c) into dev (1192720) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev   #13081      +/-   ##
============================================
- Coverage     39.22%   39.21%   -0.01%     
+ Complexity     4275     4272       -3     
============================================
  Files          1069     1069              
  Lines         40311    40311              
  Branches       4634     4634              
============================================
- Hits          15812    15808       -4     
- Misses        22722    22723       +1     
- Partials       1777     1780       +3     
Impacted Files Coverage Δ
...er/master/dispatch/host/assign/RandomSelector.java 77.77% <0.00%> (-5.56%) ⬇️
...dolphinscheduler/remote/future/ResponseFuture.java 81.96% <0.00%> (-1.64%) ⬇️
...r/plugin/task/sqoop/parameter/SqoopParameters.java 51.19% <0.00%> (-1.20%) ⬇️
...e/dolphinscheduler/remote/NettyRemotingClient.java 52.08% <0.00%> (-0.70%) ⬇️

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

@sonarcloud
Copy link

sonarcloud bot commented Dec 2, 2022

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

No Coverage information No Coverage information
No Duplication information No Duplication information

@darrkz
Copy link
Contributor Author

darrkz commented Dec 2, 2022

hi @darkz1984 , same as #13080 ?

# 13080 is for MySQL database

@darrkz darrkz changed the title Update dolphinscheduler_ddl.sql Update dolphinscheduler_ddl.sql - for PostGreSQL Dec 2, 2022
@darrkz darrkz changed the title Update dolphinscheduler_ddl.sql - for PostGreSQL Update dolphinscheduler_ddl.sql - for PostgreSQL Dec 2, 2022
Copy link
Contributor

@davidzollo davidzollo left a comment

Choose a reason for hiding this comment

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

+1

@davidzollo davidzollo modified the milestones: 3.0.2, 3.1.2 Dec 3, 2022
@davidzollo davidzollo added 3.1.x for 3.1.x version improvement make more easy to user or prompt friendly labels Dec 3, 2022
@EricGao888
Copy link
Member

Same comment here as #12826 (comment)

@davidzollo davidzollo merged commit fa3e2a5 into apache:dev Dec 3, 2022
ruanwenjun added a commit that referenced this pull request Dec 6, 2022
@zhuangchong zhuangchong modified the milestones: 3.1.2, 3.2.0 Dec 7, 2022
ruanwenjun added a commit that referenced this pull request Dec 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.1.x for 3.1.x version backend improvement make more easy to user or prompt friendly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants