dentiny opened a new pull request, #690:
URL: https://github.com/apache/arrow-rs-object-store/pull/690

   # Which issue does this PR close?
   
   Closes https://github.com/apache/arrow-rs-object-store/issues/654
   
   # Rationale for this change
    
   Disclaimer: this PR tries to solve the same problem as 
https://github.com/apache/arrow-rs-object-store/pull/609, but did a different 
way: I tried to keep the bytes since I observe (for my usage personally) most 
of returned range is a superset for requested range. 
   Feel free to close it if maintainers think it's a duplicate.
   
   Currently retry stream returns returned body directly without checking the 
range header.
   
https://github.com/apache/arrow-rs-object-store/blob/e89f62b6508f6f65873ce6a5017cd0d5d7395184/src/client/get.rs#L247
   
   # What changes are included in this PR?
   
   Since retried response could contain ranges which don't match with requested 
one, so I tried to keep useful parts if applicable.
   
   # 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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to