Seokyun-Ha commented on PR #34071:
URL: https://github.com/apache/airflow/pull/34071#issuecomment-1736824410
Hello, guys, I and @kyeonghoon-kim discussed about this active_cluster and
this PR. We noticed this PR has too many changes on DatabricksHook and other.
So, we decided to separate this PR into smaller ones.
Kind of changes
- `RunState`, `ClusterState` and `get_cluster_state()`
- We will open PR for this as soon as possible
- `active_cluster()` method
- This method can be replaced into Airflow Sensor. We will implement a
sensor for Databricks Cluster State in future PR. It will monitor cluster state
until the desired state met.
- `async_get_xxx()`
- This change are dropped, because they make breaking changes.
Thanks for your kind and sincere reviews 🙏 : We will close this PR after we
make a new separated PR 😄
cc. @Lee-W @hussein-awala @alexott
--
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]
