[GitHub] [airflow] vincbeck commented on pull request #30500: Add deferrable BatchOperator

["vincbeck (via GitHub)" <gi...@apache.org>]

vincbeck commented on PR #30500:
URL: https://github.com/apache/airflow/pull/30500#issuecomment-1516888768

   There are multiple work in parallel to add deferrable operators in Amazon provider package. I do not think there is one better way than the other but I definitely think we should all follow the same pattern for consistency purposes. @syedahsn has been working on adding support for deferrable operators in Amazon provider package in #30032. For instance, he does not create a separate hook for all async operations which I agree with because, I think, having two hooks per service (one for sync operations, one for async operations) would be adding too many files and end up making the amazon provider package harder to maintain.
   
   As a result, I would try to follow the pattern initiated in #30032 while in the meantime we should also try to merge ASAP #30032 to avoid this kind of situation again.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org