Skip to content

fix #2442 when the resource name contains '$',need translate it to '\$'#2524

Merged
qiaozhanwei merged 5 commits intoapache:devfrom
lgcareer:dev-updateHadoopUtil
Apr 27, 2020
Merged

fix #2442 when the resource name contains '$',need translate it to '\$'#2524
qiaozhanwei merged 5 commits intoapache:devfrom
lgcareer:dev-updateHadoopUtil

Conversation

@lgcareer
Copy link
Copy Markdown
Contributor

What is the purpose of the pull request

fix #2442 when the resource name contains '$',need translate it to '$'

Brief change log

fix #2442 when the resource name contains '$',need translate it to '$'

@codecov-io
Copy link
Copy Markdown

codecov-io commented Apr 26, 2020

Codecov Report

Merging #2524 into dev will increase coverage by 0.02%.
The diff coverage is 66.66%.

Impacted file tree graph

@@             Coverage Diff              @@
##                dev    #2524      +/-   ##
============================================
+ Coverage     35.92%   35.94%   +0.02%     
- Complexity     2406     2408       +2     
============================================
  Files           424      424              
  Lines         19589    19595       +6     
  Branches       2390     2390              
============================================
+ Hits           7037     7044       +7     
- Misses        11926    11927       +1     
+ Partials        626      624       -2     
Impacted Files Coverage Δ Complexity Δ
...dolphinscheduler/api/service/ResourcesService.java 57.06% <66.66%> (+0.28%) 67.00 <0.00> (+1.00)
...he/dolphinscheduler/common/thread/ThreadUtils.java 68.25% <0.00%> (-3.18%) 13.00% <0.00%> (-1.00%)
...e/dolphinscheduler/remote/NettyRemotingClient.java 53.95% <0.00%> (+2.87%) 11.00% <0.00%> (+2.00%)

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 c50ad12...0b263d2. Read the comment docs.

Copy link
Copy Markdown
Contributor

@qiaozhanwei qiaozhanwei left a comment

Choose a reason for hiding this comment

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

+1

@sonarqubecloud
Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

75.0% 75.0% Coverage
0.0% 0.0% Duplication

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.

[BUG] update the directory of resource is error

4 participants