Tom-Newton opened a new pull request, #49049:
URL: https://github.com/apache/arrow/pull/49049

   ### Rationale for this change
   Prevent bugs similar to https://github.com/apache/arrow/issues/49043
   
   ### What changes are included in this PR?
   - Implement `SkipStartingEmptyPages` for various types of PagedResponses 
used in the `AzureFileSystem`.
   - Apply `SkipStartingEmptyPages` on the response from every list operation 
that returns a paged response.
    
   ### Are these changes tested?
   Ran the tests in the codebase including the ones that need to connect to 
real blob storage. This makes me fairly confident that I haven't introduced a 
regression.
   
   The only reproduce I've found involves reading a production Azure blob 
storage account. With this I've tested that this PR solves 
https://github.com/apache/arrow/issues/49043, but I haven't been able to 
reproduce it in any checked in tests.
   
   ### Are there any user-facing changes?
   Some low probability bugs will be gone. No interface changes. 


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