thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-09-20 00:30:49 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #49737 from ClickHouse/fix_lwd_and_object

Browse Source 
Ignore LWD column in checkPartDynamicColumns
This commit is contained in:
Alexander Gololobov 2023-05-12 12:20:07 +02:00 committed by GitHub
commit 7c7565f094
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 18 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/Storages/MergeTree/MergeTreeData.cpp
View File

@ -3608,6 +3608,9 @@ void MergeTreeData::checkPartDynamicColumns(MutableDataPartPtr & part, DataParts
const auto & part_columns = part->getColumns();
for (const auto & part_column : part_columns)
{
if (part_column.name == LightweightDeleteDescription::FILTER_COLUMN.name)
continue;
auto storage_column = columns.getPhysical(part_column.name);
if (!storage_column.type->hasDynamicSubcolumns())
continue;

1
tests/queries/0_stateless/02352_lightweight_delete_and_object_column.reference Normal file
View File

@ -0,0 +1 @@
99

14
tests/queries/0_stateless/02352_lightweight_delete_and_object_column.sql Normal file
View File

@ -0,0 +1,14 @@
DROP TABLE IF EXISTS t_obj SYNC;
SET allow_experimental_object_type=1;
CREATE TABLE t_obj(id Int32, name Object('json')) ENGINE = MergeTree() ORDER BY id;
INSERT INTO t_obj select number, '{"a" : "' || toString(number) || '"}' FROM numbers(100);
DELETE FROM t_obj WHERE id = 10;
SELECT COUNT() FROM t_obj;
DROP TABLE t_obj SYNC;