flytekit icon indicating copy to clipboard operation
flytekit copied to clipboard

Flytekit Rust entrypoint

Open kumare3 opened this issue 1 year ago • 16 comments

Tracking issue

  • Faster performance - faster startup. In limited testing more than 3.8x speedup
  • Smaller footprint - our goal is to remove object-store dependencies and replace with rust
  • Also remove dependency on grpc in python and only replace with rust

This also introduces rust more wholistically in flytekit

kumare3 avatar Mar 29 '24 05:03 kumare3

Codecov Report

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

Project coverage is 95.71%. Comparing base (d32ce8f) to head (d34ce56). Report is 27 commits behind head on master.

:exclamation: Current head d34ce56 differs from pull request most recent head 06752b3. Consider uploading reports for the commit 06752b3 to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #2307       +/-   ##
===========================================
+ Coverage   75.95%   95.71%   +19.75%     
===========================================
  Files         181       19      -162     
  Lines       18295      560    -17735     
  Branches     3788        0     -3788     
===========================================
- Hits        13896      536    -13360     
+ Misses       3807       24     -3783     
+ Partials      592        0      -592     

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

codecov[bot] avatar Mar 29 '24 05:03 codecov[bot]

@vlad-ivanov-name thank you for all the comments. As you can tell I am complete newbie and I will definitely try to adapt all your suggestions. If you think you can take over the PR and improve it - please feel free too :) I would love to hand it over

kumare3 avatar Apr 04 '24 17:04 kumare3