From b1ed56c10b46b56a6e05c4f388c5e1a4bff28d26 Mon Sep 17 00:00:00 2001 From: Jake Smith Date: Wed, 11 Dec 2024 18:36:42 +0000 Subject: [PATCH] HPCC-33116 review changes Signed-off-by: Jake Smith --- dali/base/dadfs.cpp | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/dali/base/dadfs.cpp b/dali/base/dadfs.cpp index fe390b2aa2c..5841df604f8 100644 --- a/dali/base/dadfs.cpp +++ b/dali/base/dadfs.cpp @@ -13811,10 +13811,13 @@ static bool doesPhysicalMatchMeta(IPropertyTree &partProps, IFile &iFile, offset { // NB: temporary workaround for 'narrow' files publishing extra empty parts with the wrong @compressedSize(0) // causing a new check introduced in HPCC-33064 to be hit (fixed in HPCC-33113, but will continue to affect exiting files) - unsigned __int64 size = partProps.getPropInt64("@size", (unsigned __int64)-1); - unsigned __int64 compressedSize = partProps.getPropInt64("@compressedSize", (unsigned __int64)-1); + unsigned __int64 size = partProps.getPropInt64("@size", unknownFileSize); + unsigned __int64 compressedSize = partProps.getPropInt64("@compressedSize", unknownFileSize); if ((0 == size) && (0 == compressedSize)) + { + actualSize = unknownFileSize; return true; + } if (expectedSize != unknownFileSize) { @@ -13822,6 +13825,9 @@ static bool doesPhysicalMatchMeta(IPropertyTree &partProps, IFile &iFile, offset if (actualSize != expectedSize) return false; } + else + actualSize = unknownFileSize; + return true; }