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
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
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
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
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
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 ?)
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
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
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