The GitHub Actions job "Tests" on airflow.git has succeeded. Run started by GitHub user josh-fell (triggered by josh-fell).
Head commit for run: b50c4cfa78906dc82bc42ddec799557c0d0d409f / Josh Fell <[email protected]> Use `cached_property` for hook in SparkKubernetesSensor Best practice to is make constructors of operators/sensors as simple as possible. Currently this sensor directly builds a KubernetesHook object in its construction. Yes, there are db or external calls when building the hook object itself, but: 1. if this hook changes for any reason, these types calls might sneak in 2. doesn't allow for templating of `kubernetes_conn_id` as brought up in [Airflow Slack](https://apache-airflow.slack.com/archives/CCQ7EGB1P/p1693904864661079) Report URL: https://github.com/apache/airflow/actions/runs/6086092375 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
