Re: [PR] Allow executors to be specified with only the class name of the Executor [airflow]

2024-06-09 Thread via GitHub
potiuk commented on PR #40131: URL: https://github.com/apache/airflow/pull/40131#issuecomment-2156444650 Absolutely -- 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 unsub

Re: [PR] Allow executors to be specified with only the class name of the Executor [airflow]

2024-06-09 Thread via GitHub
o-nikolas commented on PR #40131: URL: https://github.com/apache/airflow/pull/40131#issuecomment-2156405318 > I guess it should be here https://airflow.apache.org/docs/apache-airflow/stable/core-concepts/executor/index.html - but maybe it's not yet done at all for multi-executors (@o-nikola

Re: [PR] Allow executors to be specified with only the class name of the Executor [airflow]

2024-06-08 Thread via GitHub
potiuk merged PR #40131: URL: https://github.com/apache/airflow/pull/40131 -- 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.a

Re: [PR] Allow executors to be specified with only the class name of the Executor [airflow]

2024-06-08 Thread via GitHub
syedahsn commented on PR #40131: URL: https://github.com/apache/airflow/pull/40131#issuecomment-2156185324 Yes, the plan is to have documentation for multiple executors in a [separate PR](https://github.com/apache/airflow/pull/37635#issue-2149928560), once we've delivered the major features

Re: [PR] Allow executors to be specified with only the class name of the Executor [airflow]

2024-06-08 Thread via GitHub
potiuk commented on PR #40131: URL: https://github.com/apache/airflow/pull/40131#issuecomment-2156153221 Generally Approved - just want to make sure that docs will be updated in a separate PR -- This is an automated message from the Apache Git Service. To respond to the message, please lo

Re: [PR] Allow executors to be specified with only the class name of the Executor [airflow]

2024-06-08 Thread via GitHub
potiuk commented on PR #40131: URL: https://github.com/apache/airflow/pull/40131#issuecomment-2156144284 I guess it should be here https://airflow.apache.org/docs/apache-airflow/stable/core-concepts/executor/index.html - but maybe it's not yet done at all for multi-executors (@o-nikolas ?)

Re: [PR] Allow executors to be specified with only the class name of the Executor [airflow]

2024-06-08 Thread via GitHub
syedahsn commented on PR #40131: URL: https://github.com/apache/airflow/pull/40131#issuecomment-2156140040 That's a good point, I missed that, thanks! I updated the doc string. Is there another place that needs to be updated? -- This is an automated message from the Apache Git Service. To

Re: [PR] Allow executors to be specified with only the class name of the Executor [airflow]

2024-06-08 Thread via GitHub
potiuk commented on PR #40131: URL: https://github.com/apache/airflow/pull/40131#issuecomment-2155940480 Should we also update the documentation about this one? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

[PR] Allow executors to be specified with only the class name of the Executor [airflow]

2024-06-07 Thread via GitHub
syedahsn opened a new pull request, #40131: URL: https://github.com/apache/airflow/pull/40131 Currently, specifying executors is done by an alias, or the full module path of the Executor. This change allows users to also use the class name of the executor to specify which executor to run th