amoghrajesh commented on code in PR #49747:
URL: https://github.com/apache/airflow/pull/49747#discussion_r2059100670
##########
airflow-core/src/airflow/executors/local_executor.py:
##########
@@ -108,6 +108,10 @@ def _execute_work(log: logging.Logger, workload:
workloads.ExecuteTask) -> None:
from airflow.sdk.execution_time.supervisor import supervise
setproctitle(f"airflow worker -- LocalExecutor: {workload.ti.id}")
+
+ base_url = conf.get("api", "base_url", fallback="/")
+ default_execution_api_server = f"{base_url.rstrip('/')}/execution/"
Review Comment:
Can we just add a property or extract this onto the base executor or
somewhere relevant?
Too many repetitions. It will be easy to miss in case of a refactor.
--
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]
