diff options
| author | Dan Engelbrecht <[email protected]> | 2024-05-17 09:56:10 +0200 |
|---|---|---|
| committer | GitHub Enterprise <[email protected]> | 2024-05-17 09:56:10 +0200 |
| commit | d8d8378e83ef88780349157d8cb1d428bf015f10 (patch) | |
| tree | 7bcd9eaaaad6c434f890e68cc671a7b7eb35e40c /src/zenstore/cache/cacherpc.cpp | |
| parent | 5.5.2-pre2 (diff) | |
| download | zen-d8d8378e83ef88780349157d8cb1d428bf015f10.tar.xz zen-d8d8378e83ef88780349157d8cb1d428bf015f10.zip | |
safer partial requests (#82)
* fix partial cache miss logging
* Handle zero size partial requests safer
Diffstat (limited to 'src/zenstore/cache/cacherpc.cpp')
| -rw-r--r-- | src/zenstore/cache/cacherpc.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/zenstore/cache/cacherpc.cpp b/src/zenstore/cache/cacherpc.cpp index f12cee923..d56ceb98e 100644 --- a/src/zenstore/cache/cacherpc.cpp +++ b/src/zenstore/cache/cacherpc.cpp @@ -1662,8 +1662,7 @@ CacheRpcHandler::WriteGetCacheChunksResponse([[maybe_unused]] const CacheRequest } } - CompressedBuffer PartialValue = - Request.Value.GetRange(Request.RequestedOffset, Request.RequestedSize == 0 ? 1u : Request.RequestedSize); + CompressedBuffer PartialValue = Request.Value.GetRange(Request.RequestedOffset, Request.RequestedSize); if (PartialValue) { @@ -1684,10 +1683,10 @@ CacheRpcHandler::WriteGetCacheChunksResponse([[maybe_unused]] const CacheRequest else { ZEN_WARN("GETCACHECHUNKS MISS (rawsize: {}, offset: {}, size: {}) - '{}/{}/{}/{}' in {}", - Namespace, Request.Value.DecodeRawSize(), Request.RequestedOffset, Request.RequestedSize, + Namespace, Request.Key->Key.Bucket, Request.Key->Key.Hash, Request.Key->ValueId, |