You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/08/20 07:07:37 UTC

[GitHub] [airflow] potiuk edited a comment on pull request #10421: SkipMixin: Add missing session.commit() and test

potiuk edited a comment on pull request #10421:
URL: https://github.com/apache/airflow/pull/10421#issuecomment-677384526


   The reason is this: https://github.com/apache/airflow/blob/master/airflow/models/xcom.py#L101 . The xcom_push method performs "expunge_all()" which removes all modified object from the session. Not sure why  though :)
   
   @kaxill  - I believe it''s not a case for 1.10.12rc4, but we should definitely merge it if there are other changes. :)


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