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

2023-03-03 Thread via GitHub
o-nikolas commented on PR #29609: URL: https://github.com/apache/airflow/pull/29609#issuecomment-1454099713 @potiuk @dimberman @uranusjr Sorry again for the confusion, but this PR is ready again for review :pray: -- This is an automated message from the Apache Git Service. To

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

2023-03-03 Thread via GitHub
o-nikolas commented on PR #29609: URL: https://github.com/apache/airflow/pull/29609#issuecomment-1453982931 Sorry for the confusion, but this code is still needed after all. I thought things had changed in a way that they haven't. I'll update this PR today -- This is an automated

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

2023-03-03 Thread via GitHub
o-nikolas commented on PR #29609: URL: https://github.com/apache/airflow/pull/29609#issuecomment-1453892080 Thanks for the feedback! I'll scrap it for now :+1: -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

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

2023-03-02 Thread via GitHub
o-nikolas commented on PR #29609: URL: https://github.com/apache/airflow/pull/29609#issuecomment-1452761063 Thoughts on the above anyone? -- 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

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

2023-02-28 Thread via GitHub
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