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

   ### Rationale for this change
   
   This change fixes the serialization/deserialization of S3FileSystem. 
Previously the retry_strategy parameter is missed in the serialization. This 
affects other distributed library like Ray to incorrectly reconstruct a 
S3FileSystem with no retry_strategy parameter in different processes.
   
   ### What changes are included in this PR?
   
   This PR includes `retry_strategy` in S3FileSystem.__reduce__ and a property 
method to allow access to retry_strategy. Since S3FileSystem is a cdef class 
but AWSDefaultRetryStrategy etc are not, `object` type is used for simplicity.
   
   ### Are these changes tested?
   
   TBA unit tests. Notably the retry_strategy property can be used to verify 
the correct reconstruction.
   
   ### Are there any user-facing changes?
   
   No.


-- 
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: github-unsubscr...@arrow.apache.org

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

Reply via email to