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

Add support for min_success_ratio for local map_task execution #1884

Merge remote-tracking branch 'origin' into Add-support-for-min_sucess…

b07546c
Select commit
Loading
Failed to load commit list.
Merged

Add support for min_success_ratio for local map_task execution #1884

Merge remote-tracking branch 'origin' into Add-support-for-min_sucess…
b07546c
Select commit
Loading
Failed to load commit list.
This check has been archived and is scheduled for deletion. Learn more about checks retention
Codecov / codecov/project succeeded Oct 30, 2023 in 0s

51.21% (+1.64%) compared to 2aaae61

View this Pull Request on Codecov

51.21% (+1.64%) compared to 2aaae61

Details

Codecov Report

Attention: 36 lines in your changes are missing coverage. Please review.

Comparison is base (2aaae61) 49.57% compared to head (b07546c) 51.21%.
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1884      +/-   ##
==========================================
+ Coverage   49.57%   51.21%   +1.64%     
==========================================
  Files         251      268      +17     
  Lines       20123    20928     +805     
  Branches     3453     3458       +5     
==========================================
+ Hits         9976    10719     +743     
- Misses       9975    10037      +62     
  Partials      172      172              
Files Coverage Δ
flytekit/core/map_task.py 30.50% <10.52%> (-1.20%) ⬇️
flytekit/core/array_node_map_task.py 32.79% <9.52%> (-1.71%) ⬇️

... and 18 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.