o-nikolas commented on PR #29609: URL: https://github.com/apache/airflow/pull/29609#issuecomment-1449246403
Interesting development: In this PR I add the notion of `is_production` to the executor interface, to represent whether or not an executor is meant to be used in production code (executors like `DebugExecutor` and `SequentialExecutor`, for example, are not intended for production use). But the only usage was in the `/run` endpoint and associated `dags.html` and @bbovenzi has removed that endpoint entirely in #29706 So shall we keep this PR which adds an attribute to the Executor interface which is now not used? Or scrap it and wait until we find another use case for it? WDYT @potiuk & @dimberman? -- 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-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
