buildbarn / bb-storage

Storage daemon, capable of storing data for the Remote Execution protocol
Apache License 2.0
142 stars 91 forks source link

Clarify the BlobAccessConfiguration.remote field comment #122

Closed mostynb closed 3 years ago

mostynb commented 3 years ago

The previous phrasing was ambiguous, since bazel supports two different cache protocols (HTTP and gRPC) and this field only supports one of them (the term "HTTP caching protocol" is used in bazel's docs).

https://docs.bazel.build/versions/main/remote-caching.html#http-caching-protocol

EdSchouten commented 3 years ago

Hey! Thanks for the patch! I used the occasion to also just rename this backend to HTTPBlobAccess, so it's more explicit.