You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "uranusjr (via GitHub)" <gi...@apache.org> on 2025/03/05 09:36:40 UTC
[I] Pull scheduler logic out of asset classes [airflow]
uranusjr opened a new issue, #47378:
URL: https://github.com/apache/airflow/issues/47378
### Body
The SDK should contain no database-related code, but currently the scheduler uses the `evaluate` function to check if an asset dependency is “ready”, which involves hitting the database in some cases. We should move it into Airflow core.
The best way to do this is probably similar to how DAGs are turned into SerializedDag. We should create a class in core Airflow to represent BaseAsset subclasses, and put logic there.
### Committer
- [x] I acknowledge that I am a maintainer/committer of the Apache Airflow project.
--
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.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Pull scheduler logic out of asset classes [airflow]
Posted by "uranusjr (via GitHub)" <gi...@apache.org>.
uranusjr closed issue #47378: Pull scheduler logic out of asset classes
URL: https://github.com/apache/airflow/issues/47378
--
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
Re: [I] Pull scheduler logic out of asset classes [airflow]
Posted by "uranusjr (via GitHub)" <gi...@apache.org>.
uranusjr commented on issue #47378:
URL: https://github.com/apache/airflow/issues/47378#issuecomment-2705781028
PR ready: #47484
--
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
Re: [I] Pull scheduler logic out of asset classes [airflow]
Posted by "JASNOTJAZZ (via GitHub)" <gi...@apache.org>.
JASNOTJAZZ commented on issue #47378:
URL: https://github.com/apache/airflow/issues/47378#issuecomment-2700870025
To fix this, we should move this logic into Airflow core by creating a new class that handles BaseAsset subclasses.
This keeps the SDK clean while ensuring database interactions happen only where they should
--
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