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 2022/07/05 04:51:37 UTC

[GitHub] [airflow] pasalkarsachin1 commented on pull request #24726: `DockerOperator` fix cli.logs giving character array instead of string

pasalkarsachin1 commented on PR #24726:
URL: https://github.com/apache/airflow/pull/24726#issuecomment-1174609512

   > > > I do wonder if there was some reason that we called got the logs again instead of using tht lines we already had. I expect not, but I wonder who might know.
   > 
   > > As per [change](https://github.com/apache/airflow/commit/2f4a3d4d4008a95fc36971802c514fef68e8a5d4) I see this is introduce by @ferruzzi
   > 
   > uranusjr wanted it that way, see discussion here: [#21175 (comment)](https://github.com/apache/airflow/pull/21175#issuecomment-1024857545)
   
   I believe the issue solved with `stringify ` API. The issue pointed in above comment is mainly because bytes are returned. Which was cases with `logs` API as well which solved by https://github.com/apache/airflow/pull/21429


-- 
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: commits-unsubscribe@airflow.apache.org

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