-
Notifications
You must be signed in to change notification settings - Fork 366
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(storage): more tweaks to ranged reads #9400
test(storage): more tweaks to ranged reads #9400
Conversation
The benchmark would perform a range read with 50% probability, but some ranged reads were "full" as the range was larger than the object. I found it hard to reason about this, so now only the range size controls whether the read is full or not. I also rediscovered a quirk of the JSON API, that it is better avoided: reading the last 0 bytes actually returns all the bytes. Maybe we should fix this in the client library itself, for now just avoid running into it.
Google Cloud Build Logs
ℹ️ NOTE: Kokoro logs are linked from "Details" below. |
Codecov Report
@@ Coverage Diff @@
## main #9400 +/- ##
=======================================
Coverage 94.60% 94.60%
=======================================
Files 1485 1485
Lines 136008 136008
=======================================
+ Hits 128672 128674 +2
+ Misses 7336 7334 -2
Continue to review full report at Codecov.
|
google/cloud/storage/benchmarks/storage_throughput_vs_cpu_benchmark.cc
Outdated
Show resolved
Hide resolved
Google Cloud Build Logs
ℹ️ NOTE: Kokoro logs are linked from "Details" below. |
The benchmark would perform a range read with 50% probability, but some
ranged reads were "full" as the range was larger than the object. I
found it hard to reason about this, so now only the range size controls
whether the read is full or not.
I also rediscovered a quirk of the JSON API, that it is better avoided:
reading the last 0 bytes actually returns all the bytes. Maybe we should
fix this in the client library itself, for now just avoid running into
it.
This change is