Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: add integration test to verify behavior of reader with effective length <= 0 #1988

Merged
merged 1 commit into from
Apr 25, 2023

Conversation

BenWhitehead
Copy link
Collaborator

StorageReadChannel is able to detect when an effective read length is <= 0 and will return EOF without sending an RPC.

If we ever return object metadata along with a read, we will need to amend this behavior so that the metadata can be resolved.

…gth <= 0

StorageReadChannel is able to detect when an effective read length is <= 0 and will return EOF without sending an RPC.

If we ever return object metadata along with a read, we will need to amend this behavior so that the metadata can be resolved.
@BenWhitehead BenWhitehead requested a review from a team as a code owner April 18, 2023 22:26
@product-auto-label product-auto-label bot added size: s Pull request size is small. api: storage Issues related to the googleapis/java-storage API. labels Apr 18, 2023
@BenWhitehead BenWhitehead added the owlbot:ignore instruct owl-bot to ignore a PR label Apr 18, 2023
@BenWhitehead BenWhitehead changed the title test: add integration to verify behavior of reader with effective length <= 0 test: add integration test to verify behavior of reader with effective length <= 0 Apr 18, 2023
@BenWhitehead BenWhitehead merged commit 46c926e into main Apr 25, 2023
@BenWhitehead BenWhitehead deleted the reader-0-length-test branch April 25, 2023 18:30
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: storage Issues related to the googleapis/java-storage API. owlbot:ignore instruct owl-bot to ignore a PR size: s Pull request size is small.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants