flyte icon indicating copy to clipboard operation
flyte copied to clipboard

Introduce userDataBucket

Open davidmirror-ops opened this issue 1 year ago • 1 comments

Tracking issue

#4291

Why are the changes needed?

Enables users to specify a separate (from metadata) Raw Data bucket in the form of a userDataBucket parameter in flyte-core.

What changes were proposed in this pull request?

How was this patch tested?

Setup process

Screenshots

Check all the applicable boxes

  • [ ] I updated the documentation accordingly.
  • [ ] All new and existing tests passed.
  • [ ] All commits are signed-off.

Related PRs

Docs link

davidmirror-ops avatar Jan 22 '24 20:01 davidmirror-ops

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (5300e1b) 58.17% compared to head (4c12d33) 58.08%. Report is 79 commits behind head on master.

:exclamation: Current head 4c12d33 differs from pull request most recent head debccf7. Consider uploading reports for the commit debccf7 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4759      +/-   ##
==========================================
- Coverage   58.17%   58.08%   -0.09%     
==========================================
  Files         626      367     -259     
  Lines       53833    32625   -21208     
==========================================
- Hits        31315    18949   -12366     
+ Misses      20010    12105    -7905     
+ Partials     2508     1571     -937     
Flag Coverage Δ
unittests 58.08% <ø> (-0.09%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Jan 22 '24 20:01 codecov[bot]