karenbraganz commented on PR #53435:
URL: https://github.com/apache/airflow/pull/53435#issuecomment-3152130914

   I was able to find a workaround for the `DetachedInstanceError` that does 
not involve eager loading. Right before the `TaskCallbackRequest` is created 
(where the error occurs), I added a condition to check whether the ti is 
detached. If it is, it will be merged into the session. I found that this 
prevents the issue from occurring without having to eager load any attributes. 
   
   I have implemented this in my latest commit. Please let me know if there are 
any objections to this workaround.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to