Daft icon indicating copy to clipboard operation
Daft copied to clipboard

feat: Merge scan tasks in flotilla

Open colin-ho opened this issue 7 months ago • 1 comments

Changes Made

Related Issues

Checklist

  • [ ] Documented in API Docs (if applicable)
  • [ ] Documented in User Guide (if applicable)
  • [ ] If adding a new documentation page, doc is added to docs/mkdocs.yml navigation
  • [ ] Documentation builds and is formatted properly (tag @/ccmao1130 for docs review)

colin-ho avatar Aug 22 '25 18:08 colin-ho

Codecov Report

:x: Patch coverage is 58.57143% with 29 lines in your changes missing coverage. Please review. :white_check_mark: Project coverage is 75.95%. Comparing base (858cfa3) to head (53ebf72). :warning: Report is 397 commits behind head on main.

Files with missing lines Patch % Lines
.../daft-distributed/src/pipeline_node/scan_source.rs 58.57% 29 Missing :warning:
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #5032      +/-   ##
==========================================
- Coverage   75.98%   75.95%   -0.03%     
==========================================
  Files         941      941              
  Lines      129461   129520      +59     
==========================================
+ Hits        98366    98377      +11     
- Misses      31095    31143      +48     
Files with missing lines Coverage Δ
.../daft-distributed/src/pipeline_node/scan_source.rs 53.71% <58.57%> (+2.89%) :arrow_up:

... and 10 files with indirect coverage changes

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov[bot] avatar Aug 22 '25 18:08 codecov[bot]