derkuci opened a new pull request #20472: URL: https://github.com/apache/airflow/pull/20472
Currently the return value from `SSHOperator.execute()` depends on whether `enable_xcom_pickling` is True or False. * If `enable_xcom_pickling=True`, a `bytes` object is returned. * otherwise, an `str` object with uuencode is returned. Later the return value may be stored in XCom and retrieved by other tasks. This confuses the "consumer" of its XCom messages. They have to also check `enable_xcom_pickling` to know if the result should first be uudecoded. Even worse, they have to know it is generated by an "SSHOperator" because other operators do not do that. Should we change the behavior to always returning a `str`, without uuencode, as the attached pull request suggests? Is that a sensible thing to do? Thanks! -- 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]
