search

astronomer / astronomer-providers

Airflow Providers containing Deferrable Operators & Sensors from Astronomer
https://astronomer-providers.rtfd.io/
Apache License 2.0
129 stars 25 forks source link

Deprecate SageMakerTrainingOperatorAsync #1463

Closed Lee-W closed 5 months ago

Lee-W commented 5 months ago

Deprecate SageMakerTrainingOperatorAsync and proxy them to their Airflow OSS provider's counterpart.

this needs to wait for the release of https://github.com/apache/airflow/pull/36685

codecov[bot] commented 5 months ago

Codecov Report

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

Comparison is base (040e724) 98.27% compared to head (605f33d) 98.24%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1463 +/- ## ========================================== - Coverage 98.27% 98.24% -0.04% ========================================== Files 91 91 Lines 4648 4609 -39 ========================================== - Hits 4568 4528 -40 - Misses 80 81 +1 ```

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

pankajkoti commented 5 months ago

Tested the DAG it worked fine with the RC, din't not encounter the issue that Wei has reported on RC Voting thread.

Screenshot 2024-02-14 at 1 50 20 PM Screenshot 2024-02-14 at 1 57 38 PM
pankajkoti commented 5 months ago

I will merge this so that we can test our regressions DAGs together on all RCs and create a separate PR later to update the versions in setup.cfg to released versions from the current RCs