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

dbms: more handy error message [#METR-2944].

Browse Source
This commit is contained in:
Alexey Milovidov 2014-10-03 03:31:18 +04:00
parent c1bb78df32
commit 1da86169d6
3 changed files with 16 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
dbms/include/DB/Core/ErrorCodes.h
View File

@ -260,6 +260,7 @@ namespace ErrorCodes
NOT_ENOUGH_BLOCK_NUMBERS,
NO_SUCH_REPLICA,
TOO_MUCH_PARTS,
REPLICA_IS_ALREADY_EXIST,
POCO_EXCEPTION = 1000,
STD_EXCEPTION,

2
dbms/include/DB/Storages/MergeTree/ReplicatedMergeTreeBlockOutputStream.h
View File

@ -125,7 +125,7 @@ public:
+ zkutil::ZooKeeper::error2string(code), ErrorCodes::UNEXPECTED_ZOOKEEPER_ERROR);
}
}
catch (zkutil::KeeperException & e)
catch (const zkutil::KeeperException & e)
{
/** Если потерялось соединение, и мы не знаем, применились ли изменения, нельзя удалять локальный кусок:
* если изменения применились, в /blocks/ появился вставленный блок, и его нельзя будет вставить снова.

17
dbms/src/Storages/StorageReplicatedMergeTree.cpp
View File

@ -271,7 +271,18 @@ void StorageReplicatedMergeTree::createReplica()
ops.push_back(new zkutil::Op::Create(replica_path + "/queue", "", zookeeper->getDefaultACL(), zkutil::CreateMode::Persistent));
ops.push_back(new zkutil::Op::Create(replica_path + "/parts", "", zookeeper->getDefaultACL(), zkutil::CreateMode::Persistent));
ops.push_back(new zkutil::Op::Create(replica_path + "/flags", "", zookeeper->getDefaultACL(), zkutil::CreateMode::Persistent));
zookeeper->multi(ops);
try
{
zookeeper->multi(ops);
}
catch (const zkutil::KeeperException & e)
{
if (e.code == ZNODEEXISTS)
throw Exception("Replica " + replica_path + " is already exist.", ErrorCodes::REPLICA_IS_ALREADY_EXIST);
throw;
}
/** Нужно изменить данные ноды /replicas на что угодно, чтобы поток, удаляющий старые записи в логе,
* споткнулся об это изменение и не удалил записи, которые мы еще не прочитали.
@ -404,7 +415,7 @@ void StorageReplicatedMergeTree::activateReplica()
{
zookeeper->multi(ops);
}
catch (zkutil::KeeperException & e)
catch (const zkutil::KeeperException & e)
{
if (e.code == ZNODEEXISTS)
throw Exception("Replica " + replica_path + " appears to be already active. If you're sure it's not, "
@ -1201,7 +1212,7 @@ void StorageReplicatedMergeTree::queueUpdatingThread()
queue_updating_event->wait();
}
catch (zkutil::KeeperException & e)
catch (const zkutil::KeeperException & e)
{
if (e.code == ZINVALIDSTATE)
restarting_event.set();