thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-21 07:01:59 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix stupid bug with settings alter

Browse Source
This commit is contained in:
alesapin 2020-02-28 15:44:50 +03:00
parent 6fb48c5e9a
commit fc18f82189
4 changed files with 22 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
dbms/src/Storages/AlterCommands.cpp
View File

@ -628,6 +628,11 @@ void AlterCommands::validate(const StorageInMemoryMetadata & metadata, const Con
ErrorCodes::NOT_FOUND_COLUMN_IN_BLOCK}; ErrorCodes::NOT_FOUND_COLUMN_IN_BLOCK};
} }
} }
else if (command.type == AlterCommand::MODIFY_SETTING)
{
if (metadata.settings_ast == nullptr)
throw Exception{"Cannot alter settings, because table engine doesn't support settings changes", ErrorCodes::BAD_ARGUMENTS};
}
} }
} }

2
dbms/tests/queries/0_stateless/00980_merge_alter_settings.sql
View File

@ -5,7 +5,7 @@ CREATE TABLE log_for_alter (
Data String Data String
) ENGINE = Log(); ) ENGINE = Log();
ALTER TABLE log_for_alter MODIFY SETTING aaa=123; -- { serverError 48 } ALTER TABLE log_for_alter MODIFY SETTING aaa=123; -- { serverError 36 }
DROP TABLE IF EXISTS log_for_alter; DROP TABLE IF EXISTS log_for_alter;

1
dbms/tests/queries/0_stateless/01089_alter_settings_old_format.reference Normal file
View File

@ -0,0 +1 @@
1

15
dbms/tests/queries/0_stateless/01089_alter_settings_old_format.sql Normal file
View File

@ -0,0 +1,15 @@
DROP TABLE IF EXISTS old_format_mt;
CREATE TABLE old_format_mt (
event_date Date,
key UInt64,
value1 UInt64,
value2 String
)
ENGINE = MergeTree(event_date, (key, value1), 8192);
ALTER TABLE old_format_mt MODIFY SETTING enable_mixed_granularity_parts = 1; --{serverError 36}
SELECT 1;
DROP TABLE IF EXISTS old_format_mt;