Skip to content

Commit 97a939a

Browse files
Avogarclaude
andcommitted
Adapt StorageMaterializedPostgreSQL to the new getColumnSizes signature
Master added a getColumnSizes(const Names &) override to StorageMaterializedPostgreSQL after this branch was created. After merging master, update that override to the new getColumnSizes(const Names &, bool) signature so it forwards calculate_subcolumn_sizes to the nested table. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent d9ea54e commit 97a939a

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

src/Storages/PostgreSQL/StorageMaterializedPostgreSQL.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -354,10 +354,10 @@ IStorage::ColumnSizeByName StorageMaterializedPostgreSQL::getColumnSizes() const
354354
}
355355

356356

357-
IStorage::ColumnSizeByName StorageMaterializedPostgreSQL::getColumnSizes(const Names & columns) const
357+
IStorage::ColumnSizeByName StorageMaterializedPostgreSQL::getColumnSizes(const Names & columns, bool calculate_subcolumn_sizes) const
358358
{
359359
if (auto nested = tryGetNested())
360-
return nested->getColumnSizes(columns);
360+
return nested->getColumnSizes(columns, calculate_subcolumn_sizes);
361361
return {};
362362
}
363363

src/Storages/PostgreSQL/StorageMaterializedPostgreSQL.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,7 @@ class StorageMaterializedPostgreSQL final : public IStorage, WithContext
107107
bool supportsOptimizationToSubcolumns() const override;
108108

109109
ColumnSizeByName getColumnSizes() const override;
110-
ColumnSizeByName getColumnSizes(const Names & columns) const override;
110+
ColumnSizeByName getColumnSizes(const Names & columns, bool calculate_subcolumn_sizes) const override;
111111

112112
std::optional<UInt64> totalRows(ContextPtr query_context) const override;
113113
std::optional<UInt64> totalBytes(ContextPtr query_context) const override;

0 commit comments

Comments
 (0)