disable partial getcachechunk responses (#19)

1 files changed, 7 insertions, 1 deletions

diff --git a/src/zenstore/cache/cacherpc.cpp b/src/zenstore/cache/cacherpc.cpp
index 27bc4b016..84ca5c5e3 100644
--- a/src/zenstore/cache/cacherpc.cpp
+++ b/src/zenstore/cache/cacherpc.cpp
@@ -1549,7 +1549,13 @@ CacheRpcHandler::WriteGetCacheChunksResponse([[maybe_unused]] const CacheRequest
 
 	using namespace cache::detail;
 
-	const bool AcceptsPartialChunks = EnumHasAnyFlags(AcceptOptions, RpcAcceptOptions::kAllowPartialCacheChunks);
+	const bool AcceptsPartialChunks = EnumHasAnyFlags(AcceptOptions, RpcAcceptOptions::kAllowPartialCacheChunks) && false;
+	// TODO: We need to redesign how we respond with partial chunks as we can get requests for multiple parts of the same chunk
+	// and we currently identify chunks by the full chunks RawHash making it impossible to distinguish between sections for partial chunks.
+	// Also, UE currently does the lookup by checking the RawHash *inside* the attachment which will break as sections of compressed
+	// buffers has a RawHash of zero (as we don't want to re-hash the section)
+	// zen side handles this as it separates the lookup hash from the hash inside the attachment, but it still can't tell two partial
+	// section from the same chunk apart.
 
 	CbPackage	   RpcResponse;
 	CbObjectWriter Writer;