thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-22 15:42:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

dbms: removed deprecated 'replica_name' configuration parameter [#METR-10202].

Browse Source
This commit is contained in:
Alexey Milovidov 2014-08-22 05:01:28 +04:00
parent 2b188ba6e0
commit aae80fa8c5
3 changed files with 2 additions and 21 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
dbms/include/DB/Interpreters/Context.h
View File

@ -96,7 +96,6 @@ struct ContextShared
ViewDependencies view_dependencies; /// Текущие зависимости
ConfigurationPtr users_config; /// Конфиг с секциями users, profiles и quotas.
InterserverIOHandler interserver_io_handler; /// Обработчик для межсерверной передачи данных.
String default_replica_name; /// Имя реплики из конфига. DEPRECATED
BackgroundProcessingPoolPtr background_pool; /// Пул потоков для фоновой работы, выполняемой таблицами.
Macros macros; /// Подстановки из конфига.
@ -238,10 +237,6 @@ public:
String getDefaultFormat() const; /// Если default_format не задан - возвращается некоторый глобальный формат по-умолчанию.
void setDefaultFormat(const String & name);
/// Имя этой реплики из конфига.
String getDefaultReplicaName() const;
void setDefaultReplicaName(const String & name);
const Macros & getMacros() const;
void setMacros(Macros && macros);

12
dbms/src/Interpreters/Context.cpp
View File

@ -10,6 +10,7 @@
#include <DB/Interpreters/Context.h>
#include <DB/Client/ConnectionPoolWithFailover.h>
namespace DB
{
@ -448,17 +449,6 @@ void Context::setDefaultFormat(const String & name)
default_format = name;
}
String Context::getDefaultReplicaName() const
{
return shared->default_replica_name;
}
void Context::setDefaultReplicaName(const String & name)
{
/// Полагаемся, что это присваивание происходит один раз при старте сервера. Если это не так, нужно использовать мьютекс.
shared->default_replica_name = name;
}
const Macros& Context::getMacros() const
{
return shared->macros;

4
dbms/src/Storages/StorageFactory.cpp
View File

@ -267,12 +267,8 @@ StoragePtr StorageFactory::get(
else
throw Exception("Replica name must be a string literal", ErrorCodes::BAD_ARGUMENTS);
if (replica_name.empty())
{
replica_name = context.getDefaultReplicaName();
if (replica_name.empty())
throw Exception("No replica name in config", ErrorCodes::NO_REPLICA_NAME_GIVEN);
}
args.erase(args.begin(), args.begin() + 2);
}