You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "atharv9017 (via GitHub)" <gi...@apache.org> on 2025/01/21 15:50:53 UTC

[I] Get Operators subclassing SkipMixin working with Task SDK [airflow]

atharv9017 opened a new issue, #45838:
URL: https://github.com/apache/airflow/issues/45838

   ### Apache Airflow version
   
   2.10.4
   
   ### If "Other Airflow 2 version" selected, which one?
   
   _No response_
   
   ### What happened?
   
   The following classes subclass airflow.models.skipmixin.SkipMixin, which directly accesses the DB:
   
   BranchMixIn (used in BaseBranchOperator, BranchPythonOperator, BranchDateTimeOperator, LatestOnlyOperator, etc.)
   BaseSensorOperator
   BranchSQLOperator
   ShortCircuitOperator
   These operators need to be refactored to work with Task SDK, as the current implementation directly accesses the database. The operations should be routed to the API server from the worker instead of the current DB access.
   
   ### What you think should happen instead?
   
   Remove the inheritance of these operators from SkipMixin.
   Route the DB-related operations to the API Server from the worker.
   
   ### How to reproduce
   
   Use any of the affected operators (e.g., BaseBranchOperator, BranchPythonOperator, etc.).
   Observe that DB operations are being performed directly by the operators, causing a dependency on DB access in the Task SDK context.
   
   ### Operating System
   
   Windows
   
   ### Versions of Apache Airflow Providers
   
   _No response_
   
   ### Deployment
   
   Official Apache Airflow Helm Chart
   
   ### Deployment details
   
   _No response_
   
   ### Anything else?
   
   This issue occurs whenever the affected operators are used, causing issues with Task SDK integration due to direct DB access.
   
   ### Are you willing to submit PR?
   
   - [x] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [x] I agree to follow this project's [Code of Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)
   


-- 
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] Get Operators subclassing SkipMixin working with Task SDK [airflow]

Posted by "boring-cyborg[bot] (via GitHub)" <gi...@apache.org>.
boring-cyborg[bot] commented on issue #45838:
URL: https://github.com/apache/airflow/issues/45838#issuecomment-2605102859

   Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for approval.
   


-- 
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] Get Operators subclassing SkipMixin working with Task SDK [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on issue #45838:
URL: https://github.com/apache/airflow/issues/45838#issuecomment-2605608884

   This is likely a bot repeating 2 of the issues created earier by @kaxil. I am reporting the bot to GitHub


-- 
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] Get Operators subclassing SkipMixin working with Task SDK [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on issue #45838:
URL: https://github.com/apache/airflow/issues/45838#issuecomment-2605616718

   Reported


-- 
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] Get Operators subclassing SkipMixin working with Task SDK [airflow]

Posted by "tirkarthi (via GitHub)" <gi...@apache.org>.
tirkarthi commented on issue #45838:
URL: https://github.com/apache/airflow/issues/45838#issuecomment-2605181960

   Is this a duplicate of https://github.com/apache/airflow/issues/45823 ?


-- 
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] Get Operators subclassing SkipMixin working with Task SDK [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk closed issue #45838: Get Operators subclassing SkipMixin working with Task SDK
URL: https://github.com/apache/airflow/issues/45838


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