eskarimov commented on PR #24099: URL: https://github.com/apache/airflow/pull/24099#issuecomment-1153536952
> Would be nice also to fix [dms example dag](https://github.com/apache/airflow/blob/4c9f7560355eefd57a29afee73bf04273e81a7e8/airflow/providers/amazon/aws/example_dags/example_dms.py) to use the operators added by this PRas you can see from the review in [#23681 (comment)](https://github.com/apache/airflow/pull/23681#discussion_r873146612) This will help also to make sure we have everything we need covered by these operators (the feature request was created from #23681 ) I agree it'd be nice, however, maybe it'd be better to make in a separate PR together with DAG refactoring making it AIP-47 compatible, in order to keep the scope of this PR not too broad? > Comments to @vincbeck comments @eskarimov ? Just replied, very nice feedback, I'm going to fix mentioned things in the upcoming days. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
