Vamsi-klu commented on PR #70088: URL: https://github.com/apache/airflow/pull/70088#issuecomment-5235779087
@eladkal pushed `727bfc3`, which covers all four of your points: the module is renamed to `warehouse.py`, the unused `WarehouseState` serializers are gone, and the docs no longer mention system tests. On defer support, my plan was to add it as a follow-up rather than fold it in here. Phase 2 adds `a_get_warehouse_state` to the hook plus a `DatabricksWarehouseStateTrigger` next to the two triggers the provider already has, keeping `wait_for_termination`, `polling_period_seconds` and `timeout` exactly as they are, so deferrable lands as an additive change rather than a parameter redesign. @moomindani reviewed that reasoning and had no objection to merging phase 1 as scoped. If you would rather see deferrable in this PR before it merges, say so and I will extend it here instead. @moomindani I also merged latest main in the same push, so the diff no longer looks like it reverts #70130 and #69442. -- 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]
