Skip to content

Conversation

@ruanwenjun
Copy link
Member

@ruanwenjun ruanwenjun commented Oct 23, 2023

Purpose of the pull request

close #15048

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

@ruanwenjun ruanwenjun force-pushed the dev_wenjun_fixMultipleSqlInWindows branch from e3fe4e1 to a46488e Compare October 23, 2023 06:00
@ruanwenjun ruanwenjun added the bug Something isn't working label Oct 23, 2023
@codecov-commenter
Copy link

codecov-commenter commented Oct 23, 2023

Codecov Report

Merging #15062 (3a9a736) into dev (a5284e4) will increase coverage by 0.01%.
The diff coverage is 38.46%.

❗ Current head 3a9a736 differs from pull request most recent head 8dc8969. Consider uploading reports for the commit 8dc8969 to get more accurate results

@@             Coverage Diff              @@
##                dev   #15062      +/-   ##
============================================
+ Coverage     37.74%   37.76%   +0.01%     
- Complexity     4588     4591       +3     
============================================
  Files          1247     1248       +1     
  Lines         44714    44711       -3     
  Branches       4913     4912       -1     
============================================
+ Hits          16877    16884       +7     
+ Misses        25959    25949      -10     
  Partials       1878     1878              
Files Coverage Δ
...scheduler/plugin/task/sql/utils/SqlSplitUtils.java 66.66% <66.66%> (ø)
...ache/dolphinscheduler/plugin/task/sql/SqlTask.java 3.49% <14.28%> (+0.55%) ⬆️

... and 2 files with indirect coverage changes

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

@ruanwenjun ruanwenjun force-pushed the dev_wenjun_fixMultipleSqlInWindows branch from 760cf3c to 8dc8969 Compare October 26, 2023 02:13
@sonarqubecloud
Copy link

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 3 Code Smells

44.4% 44.4% Coverage
0.0% 0.0% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

Copy link
Member

@SbloodyS SbloodyS left a comment

Choose a reason for hiding this comment

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

LGTM

@ruanwenjun ruanwenjun merged commit 99214d0 into apache:dev Oct 26, 2023
@ruanwenjun ruanwenjun deleted the dev_wenjun_fixMultipleSqlInWindows branch October 26, 2023 12:10
xdu-chenrj pushed a commit to xdu-chenrj/dolphinscheduler that referenced this pull request Oct 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend bug Something isn't working ready-to-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] [API] sql default separator is ;\n , but save sql content to ds database, it display ;\r\n

3 participants