Skip to content

Commit

Permalink
HPCC-33116 review changes
Browse files Browse the repository at this point in the history
Signed-off-by: Jake Smith <[email protected]>
  • Loading branch information
jakesmith committed Dec 11, 2024
1 parent 3a3e674 commit b1ed56c
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions dali/base/dadfs.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -13811,17 +13811,23 @@ 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)
{
actualSize = iFile.size();
if (actualSize != expectedSize)
return false;
}
else
actualSize = unknownFileSize;

return true;
}

Expand Down

0 comments on commit b1ed56c

Please sign in to comment.