Skip to content

Test zero-size remote file reads#984

Open
fallintoplace wants to merge 1 commit into
rapidsai:mainfrom
fallintoplace:fix-remote-zero-size-read-test
Open

Test zero-size remote file reads#984
fallintoplace wants to merge 1 commit into
rapidsai:mainfrom
fallintoplace:fix-remote-zero-size-read-test

Conversation

@fallintoplace

Copy link
Copy Markdown
Contributor

Summary

  • add Python RemoteFile.read(size=0) regression coverage for HTTP reads
  • run the zero-size reads against a no-range HTTP server so the test fails if KvikIO tries to set a range request
  • cover offsets at start-of-file and EOF while verifying the destination buffer is untouched

Notes

Current main already returns 0 in RemoteHandle::read() before setting up a range request when size is 0. This adds Python API coverage for that behavior.

Validation

  • pre-commit run --files python/kvikio/tests/test_http_io.py
  • git diff --check

Not run locally: python3 -m pytest python/kvikio/tests/test_http_io.py -k zero_size_read_returns_without_range_request -q because the system Python does not have pytest installed.

@fallintoplace fallintoplace requested a review from a team as a code owner June 23, 2026 19:38
@copy-pr-bot

copy-pr-bot Bot commented Jun 23, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant