thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-10 09:32:06 +00:00
Code Issues Packages Projects Releases Wiki Activity

adjust database count checking DatabaseCatalog::TEMPORARY_DATABASE explicitly, change 'yandex' into 'clickhouse' in config.xml

Browse Source
This commit is contained in:
Xu Jia 2024-06-10 16:45:38 +08:00
parent 446cc0542b
commit a2fd206a4d
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/Interpreters/InterpreterCreateQuery.cpp
View File

@ -149,7 +149,7 @@ BlockIO InterpreterCreateQuery::createDatabase(ASTCreateQuery & create)
{
size_t db_count = DatabaseCatalog::instance().getDatabases().size();
// there's an invisible system database _temporary_and_external_tables, so we need to subtract 1
if (db_count > 0)
if (db_count > 0 && DatabaseCatalog::instance().isDatabaseExist(DatabaseCatalog::TEMPORARY_DATABASE))
db_count--;
if (db_count >= max_db)
throw Exception(ErrorCodes::TOO_MANY_DATABASES,

4
tests/queries/0_stateless/03165_database_and_table_count_limitation_test.sh
View File

@ -19,7 +19,7 @@ TEMP_DIR=$(mktemp -d "$CUR_DIR/$(basename "$BASE").XXXXXX")
# Generate config.xml
CONFIG_FILE="$TEMP_DIR/config.xml"
cat > "$CONFIG_FILE" <<EOL
<yandex>
<clickhouse>
<logger>
<level>information</level>
<log>$TEMP_DIR/clickhouse-server.log</log>
@ -35,7 +35,7 @@ cat > "$CONFIG_FILE" <<EOL
</users_xml>
</user_directories>
</yandex>
</clickhouse>
EOL
echo "<clickhouse>