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

validate direct dictionary lifetime is unset during creation

Browse Source
This commit is contained in:
Rory Crispin 2023-04-17 17:41:02 +02:00
parent 8bc0a3a899
commit 006af1dfa1
3 changed files with 15 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/Dictionaries/getDictionaryConfigurationFromAST.cpp
View File

@ -571,11 +571,23 @@ void checkPrimaryKey(const AttributeNameToConfiguration & all_attrs, const Names
} }
void checkLifetime(const ASTCreateQuery & query)
{
if (query.dictionary -> layout && query.dictionary->layout->layout_type == "direct") {
if (query.dictionary -> lifetime)
throw Exception(
ErrorCodes::BAD_ARGUMENTS,
"'lifetime' parameter is redundant for the dictionary' of layout '{}'",
query.dictionary->layout->layout_type);
}
}
DictionaryConfigurationPtr DictionaryConfigurationPtr
getDictionaryConfigurationFromAST(const ASTCreateQuery & query, ContextPtr context, const std::string & database_) getDictionaryConfigurationFromAST(const ASTCreateQuery & query, ContextPtr context, const std::string & database_)
{ {
checkAST(query); checkAST(query);
checkLifetime(query);
AutoPtr<Poco::XML::Document> xml_document(new Poco::XML::Document()); AutoPtr<Poco::XML::Document> xml_document(new Poco::XML::Document());
AutoPtr<Poco::XML::Element> document_root(xml_document->createElement("dictionaries")); AutoPtr<Poco::XML::Element> document_root(xml_document->createElement("dictionaries"));

0
tests/queries/0_stateless/02716_create_direct_dict_with_lifetime_crash.reference Normal file
View File

3
tests/queries/0_stateless/02716_create_direct_dict_with_lifetime_crash.sql Normal file
View File

@ -0,0 +1,3 @@
CREATE TABLE IF NOT EXISTS dict_source (key UInt64, value String) ENGINE=MergeTree ORDER BY key;
CREATE DICTIONARY dict(`key` UInt64,`value` String) PRIMARY KEY key SOURCE(CLICKHOUSE(table 'dict_source')) LAYOUT(DIRECT()) LIFETIME(0); -- { serverError 36 }