boring-cyborg[bot] commented on PR #40738:
URL: https://github.com/apache/airflow/pull/40738#issuecomment-2246523554
Awesome work, congrats on your first merged pull request! You are invited to
check our [Issue Tracker](https://github.com/apache/airflow/issues) for
additional contributions.
potiuk merged PR #40738:
URL: https://github.com/apache/airflow/pull/40738
--
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
potiuk commented on PR #40738:
URL: https://github.com/apache/airflow/pull/40738#issuecomment-2246448192
> I think this is missing on SequentialExecutor as well, not sure if it's
worth fixing that here or not. Most executors use the standard task runner (and
actually in that case only the r
potiuk commented on PR #40738:
URL: https://github.com/apache/airflow/pull/40738#issuecomment-2246444558
> We're seeing this issue using the KubernetesExecutor.
K8S executor uses LocalExecutor under-the-hood, so likely this one will fix
it too. I went aheaad and fixed static-checks an
adrianrego commented on PR #40738:
URL: https://github.com/apache/airflow/pull/40738#issuecomment-2246277668
We're seeing this issue using the KubernetesExecutor.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
potiuk commented on PR #40738:
URL: https://github.com/apache/airflow/pull/40738#issuecomment-2242286334
@ErandM50 ?
--
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 unsu
potiuk commented on PR #40738:
URL: https://github.com/apache/airflow/pull/40738#issuecomment-2227491895
> I think this is missing on SequentialExecutor as well, not sure if it's
worth fixing that here or not. Most executors use the standard task runner (and
actually in that case only the r
ErandM50 opened a new pull request, #40738:
URL: https://github.com/apache/airflow/pull/40738
his PR resolves #39997
When running with local executor, get_parsing_context would return None
regardless of the parsing context. This leads to longer loading times when a
lot of dynamic DAGs ar
boring-cyborg[bot] commented on PR #40738:
URL: https://github.com/apache/airflow/pull/40738#issuecomment-2223863759
Congratulations on your first Pull Request and welcome to the Apache Airflow
community! If you have any issues or are unsure about any anything please check
our Contributors'