[GitHub] [airflow] ashb edited a comment on pull request #15581: Add optional result handler to database hooks

[GitBox <gi...@apache.org>]

ashb edited a comment on pull request #15581:
URL: https://github.com/apache/airflow/pull/15581#issuecomment-844146643


   @MatrixManAtYrService Yeah, as it stands now this hasn't been exposed to any of the Operators -- it was mostly designed as an extra tool in the db hook for _writing_ operator features.
   
   > Do we want to add the handler kwarg to the operators too? Not that the return value would be helpful there, but users could call get_current_context in the handler and push the outputs to xcom or something like that.
   
   If it's written as part of the operator, then you can make a closure to get the context vars you need.


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org