#1411 Pluck retry from flytekit and into sagemaker Pull request opened by eapolinario Signed-off-by: Eduardo Apolinario eapolinario@users.noreply.github.com TL;DR

retry

is only used in the sagemaker plugin - it should no longer be a flytekit dependence Type ☑︎ Bug Fix ☐ Feature ☐ Plugin Are all requirements met? ☐ Code completed ☐ Smoke tested ☐ Unit tests added ☐ Code documentation added ☐ Any pending items have an associated Issue Complete description

retry

is only used in the sagemaker flytekit plugin, which means that we only need

retry

there. We're going to release a minor version of flytekit and in a follow up PR we can then update the dependence in the sagemaker plugin's requirements. Tracking Issue flyteorg/flyte#3052 Follow-up issue NA flyteorg/flytekit GitHub Actions: build-plugins (3.10, flytekit-vaex) GitHub Actions: build-plugins (3.10, flytekit-sqlalchemy) GitHub Actions: build-plugins (3.10, flytekit-spark) GitHub Actions: build-plugins (3.10, flytekit-snowflake) GitHub Actions: build-plugins (3.10, flytekit-ray) GitHub Actions: build-plugins (3.10, flytekit-hive) GitHub Actions: build-plugins (3.9, flytekit-whylogs) GitHub Actions: build-plugins (3.9, flytekit-vaex) GitHub Actions: build-plugins (3.9, flytekit-sqlalchemy) GitHub Actions: build-plugins (3.9, flytekit-spark) GitHub Actions: build-plugins (3.9, flytekit-snowflake) GitHub Actions: build-plugins (3.9, flytekit-ray) GitHub Actions: build-plugins (3.9, flytekit-polars) GitHub Actions: build-plugins (3.9, flytekit-papermill) ✅ 16 other checks have passed 16/30 successful checks