dolphinscheduler icon indicating copy to clipboard operation
dolphinscheduler copied to clipboard

[DSIP-13][Feature]local_resource_plugin

Open xdu-chenrj opened this issue 3 years ago • 1 comments

Purpose of the pull request

This pull request adds a local resource plug-in feature #10911

Brief change log

 Added local resource plug-ins
 Added testing of local resource plug-ins
 Added some tests for task and shell
 Added documentation on how to use local resource plug-ins
 Added documentation on how to develop resource plug-ins

Verify this pull request

This change added tests and can be verified as follows:

 Added testing of local resource plug-ins
 Added some tests for task and shell
 Manually verified the change by testing locally

xdu-chenrj avatar Aug 08 '22 16:08 xdu-chenrj

@xdu-chenrj Please associate the issue.

SbloodyS avatar Aug 09 '22 01:08 SbloodyS

Codecov Report

Merging #11360 (fed8ef9) into dev (d8d5d39) will decrease coverage by 0.00%. The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev   #11360      +/-   ##
============================================
- Coverage     39.65%   39.65%   -0.01%     
+ Complexity     4695     4693       -2     
============================================
  Files           993      993              
  Lines         37698    37698              
  Branches       4199     4199              
============================================
- Hits          14951    14948       -3     
- Misses        21178    21179       +1     
- Partials       1569     1571       +2     
Impacted Files Coverage Δ
...er/master/dispatch/host/assign/RandomSelector.java 77.77% <0.00%> (-5.56%) :arrow_down:
...dolphinscheduler/remote/future/ResponseFuture.java 81.96% <0.00%> (-1.64%) :arrow_down:
...r/plugin/task/sqoop/parameter/SqoopParameters.java 55.12% <0.00%> (-1.29%) :arrow_down:

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

codecov-commenter avatar Aug 12 '22 04:08 codecov-commenter

I approval the CI run

zhongjiajie avatar Aug 14 '22 04:08 zhongjiajie

I fix some CI error in 61176b5

zhongjiajie avatar Aug 14 '22 10:08 zhongjiajie

This error may be due to your specific the not found path is windows path, and the value of string is not like the *unix like, we can use fuzzy matching for the error raise in fd37037

zhongjiajie avatar Aug 15 '22 07:08 zhongjiajie

This error may be due to your specific the not found path is windows path, and the value of string is not like the *unix like, we can use fuzzy matching for the error raise in fd37037

I'm still thinking about how to output the path correctly. That's wonderful.

xdu-chenrj avatar Aug 15 '22 16:08 xdu-chenrj

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

sonarqubecloud[bot] avatar Sep 06 '22 11:09 sonarqubecloud[bot]