thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-29 19:12:03 +00:00
Code Issues Packages Projects Releases Wiki Activity

dbms: fixed race condition in SingleBarrier [#METR-18510]

Browse Source
This commit is contained in:
Alexey Arno 2016-09-01 11:58:49 +03:00
parent 18930e5bb5
commit b62b499a9c
1 changed files with 16 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
libs/libzkutil/src/SingleBarrier.cpp
View File

@ -41,6 +41,22 @@ SingleBarrier::SingleBarrier(GetZooKeeper get_zookeeper_, const std::string & pa
if ((code != ZOK) && (code != ZNODEEXISTS)) if ((code != ZOK) && (code != ZNODEEXISTS))
throw KeeperException{code}; throw KeeperException{code};
if (code == ZNODEEXISTS)
{
/// This barrier is already being created by another node.
/// Wait until it has been fully created so that it is usable.
if (!zookeeper->exists(path + "/lock", nullptr, event))
{
do
{
abortIfRequested();
}
while (!event->tryWait(wait_duration));
return;
}
}
/// List of tokens. /// List of tokens.
code = zookeeper->tryCreate(path + "/tokens", "", CreateMode::Persistent); code = zookeeper->tryCreate(path + "/tokens", "", CreateMode::Persistent);
if ((code != ZOK) && (code != ZNODEEXISTS)) if ((code != ZOK) && (code != ZNODEEXISTS))