shashbha14 opened a new pull request, #48896:
URL: https://github.com/apache/arrow/pull/48896

   ### Rationale for this change
   The 
[download_rc_binaries.py](cci:7://file:///c:/Users/ishwet/arrow/dev/release/download_rc_binaries.py:0:0-0:0)
 script fails with a `TypeError` when a download fails and it tries to check if 
the error is related to OpenSSL.
   
   ### What changes are included in this PR?
   `subprocess.Popen().communicate()` returns bytes objects, but the code was 
comparing `stderr` (bytes) with a string `"OpenSSL"`. Changed to use bytes 
literal `b"OpenSSL"` for proper comparison.
   
   ### Are these changes tested?
   The fix is straightforward - using bytes literal instead of string literal 
for comparison with bytes object.
   
   ### Are there any user-facing changes?
   No.
   
   Closes #48853


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

Reply via email to