dolphinscheduler icon indicating copy to clipboard operation
dolphinscheduler copied to clipboard

[Fix-11197] [Master] Dependency detection bug

Open brave-lee opened this issue 3 years ago • 4 comments

closed by #11197

brave-lee avatar Jul 29 '22 08:07 brave-lee

Codecov Report

Merging #11204 (d1be54c) into dev (17a9dd2) will decrease coverage by 0.07%. The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                dev   #11204      +/-   ##
============================================
- Coverage     39.79%   39.71%   -0.08%     
+ Complexity     4697     4695       -2     
============================================
  Files           993      992       -1     
  Lines         37662    37755      +93     
  Branches       4186     4204      +18     
============================================
+ Hits          14986    14994       +8     
- Misses        21109    21194      +85     
  Partials       1567     1567              
Impacted Files Coverage Δ
...cheduler/server/master/utils/DependentExecute.java 0.00% <0.00%> (ø)
...nscheduler/service/process/ProcessServiceImpl.java 31.71% <0.00%> (-0.16%) :arrow_down:
...er/master/dispatch/host/assign/RandomSelector.java 77.77% <0.00%> (-5.56%) :arrow_down:
...server/master/runner/task/CommonTaskProcessor.java 4.54% <0.00%> (-0.22%) :arrow_down:
...olphinscheduler/server/registry/HeartBeatTask.java 0.00% <0.00%> (ø)
...duler/common/lifecycle/ServerLifeCycleManager.java 0.00% <0.00%> (ø)
...lugin/task/api/parameters/DependentParameters.java 0.00% <0.00%> (ø)
...uler/plugin/task/api/model/DependentTaskModel.java
...e/dolphinscheduler/remote/NettyRemotingClient.java 52.77% <0.00%> (+0.69%) :arrow_up:
... and 1 more

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

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

Any new development?

lordk911 avatar Sep 20 '22 03:09 lordk911

@SbloodyS Why this PR hung up such a long time, I think dep bug is important to be repair. or there is any other PR to resolve this problem? thanks.

lordk911 avatar Sep 22 '22 01:09 lordk911

refer #12219, I think it need new pr to solve, someone want to contribute ?

brave-lee avatar Oct 14 '22 03:10 brave-lee

refer #12219, I think it need new pr to solve, someone want to contribute ?

#12219 has no code changed but only doc , anything more need to do for is PR?

lordk911 avatar Oct 17 '22 09:10 lordk911