[GitHub] [airflow] o-nikolas commented on pull request #29609: Decouple production executor warning in dags UI

["o-nikolas (via GitHub)" <gi...@apache.org>]

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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org