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/01/17 19:03:22 UTC
[GitHub] [airflow] potiuk edited a comment on issue #20901: Bytes cast to String in `airflow.providers.google.cloud.transfers.gcs_to_local.GCSToLocalFilesystemOperator` ~142
potiuk edited a comment on issue #20901:
URL: https://github.com/apache/airflow/issues/20901#issuecomment-1014819190
Ah. I see - this actually converts the value of the bytes to "nice string" representation: https://docs.python.org/3/library/stdtypes.html#boolean-operations-and-or-not
> Passing a bytes object to str() without the encoding or errors arguments falls under the first case of returning the informal string representation (see also the -b command-line option to Python). For example:
```
str(b'Zoot!')
"b'Zoot!'"
```
So those are not "bytes" but string containing `b'` :scream:
Yeah. Agree it's a bug then. Would you like to fix it maybe? What do you think should be the fix @danneaves-ee ?
--
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