Skip to content

Commit b84dcef

Browse files
authored
Merge pull request ClickHouse#79425 from ClickHouse/davenger-patch-3
Handle the case when client returns more children than requested
2 parents b985a71 + 11263c2 commit b84dcef

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/Disks/ObjectStorages/AzureBlobStorage/AzureObjectStorage.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -189,7 +189,7 @@ void AzureObjectStorage::listObjects(const std::string & path, RelativePathsWith
189189

190190
if (max_keys)
191191
{
192-
size_t keys_left = max_keys - children.size();
192+
ssize_t keys_left = static_cast<ssize_t>(max_keys) - children.size();
193193
if (keys_left <= 0)
194194
break;
195195
options.PageSizeHint = keys_left;

src/Disks/ObjectStorages/S3/S3ObjectStorage.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -275,7 +275,7 @@ void S3ObjectStorage::listObjects(const std::string & path, RelativePathsWithMet
275275

276276
if (max_keys)
277277
{
278-
size_t keys_left = max_keys - children.size();
278+
ssize_t keys_left = static_cast<ssize_t>(max_keys) - children.size();
279279
if (keys_left <= 0)
280280
break;
281281
request.SetMaxKeys(static_cast<int>(keys_left));

0 commit comments

Comments
 (0)