turbaszek commented on a change in pull request #13536:
URL: https://github.com/apache/airflow/pull/13536#discussion_r556329300
##########
File path: tests/providers/docker/operators/test_docker.py
##########
@@ -244,7 +244,7 @@ def test_execute_xcom_behavior(self):
xcom_push_result = xcom_push_operator.execute(None)
no_xcom_push_result = no_xcom_push_operator.execute(None)
- self.assertEqual(xcom_push_result, b'container log')
+ self.assertEqual(xcom_push_result, 'container log')
Review comment:
@AlessioM I think we should merge this PR which fixes the operator -
there's no problem with current XCom implementation.
As per breaking change we can do this - we just need an info about it in
ADDITIONAL_INFO.md in docker provider.
----------------------------------------------------------------
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:
[email protected]
