You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "erichanwang (via GitHub)" <gi...@apache.org> on 2026/08/09 21:35:00 UTC

[PR] GH-50665: [Python] Assert s3fs selector result count [arrow]

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

   ### Rationale for this change
   
   The recursive selector branch for the fsspec S3 backend evaluated `len(infos) == 4` without asserting the result. As a result, an incorrect number of returned entries could pass the test unnoticed.
   
   ### What changes are included in this PR?
   
   - Assert that the recursive s3fs selector returns four entries.
   
   Fixes #50665.
   
   ### Are these changes tested?
   
   Yes. The modified test file passes Python syntax compilation, and `git diff --check` passes. The focused runtime test could not run locally because this checkout does not have the pyarrow test dependencies installed.
   
   ### Are there any user-facing changes?
   
   No. This change only strengthens an existing Python filesystem test.
   
   * GitHub Issue: #50665
   


-- 
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-unsubscribe@arrow.apache.org

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


Re: [PR] GH-50665: [Python] Assert s3fs selector result count [arrow]

Posted by "erichanwang (via GitHub)" <gi...@apache.org>.
erichanwang closed pull request #50836: GH-50665: [Python] Assert s3fs selector result count
URL: https://github.com/apache/arrow/pull/50836


-- 
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-unsubscribe@arrow.apache.org

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