Skip to content

Commit

Permalink
fixup! Add support for transferring compressed blobs via ByteStream
Browse files Browse the repository at this point in the history
Add a note about ReadRequest.read_limit referring to the compressed data
and warning clients not to assume that this value is <= the uncompressed
data size.
  • Loading branch information
mostynb committed Nov 24, 2020
1 parent 50148f6 commit 36dc185
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion build/bazel/remote/execution/v2/remote_execution.proto
Original file line number Diff line number Diff line change
Expand Up @@ -281,7 +281,10 @@ service ActionCache {
// surfacing an error to the user.
//
// Note that when reading compressed blobs, the `ReadRequest.read_offset`
// refers to the offset in the uncompressed form of the blob.
// refers to the offset in the uncompressed form of the blob, and the
// `ReadRequest.read_limit` refers to the offset in the specific compressed
// form that the server provides. Clients SHOULD NOT assume that the compressed
// form is the same or smaller size than the uncompressed form.
//
// Servers MAY use any compression level they choose, including different
// levels for different blobs (e.g. choosing a level designed for maximum
Expand Down

0 comments on commit 36dc185

Please sign in to comment.