Apache
/
druid
mirror of https://github.com/apache/druid.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Add comment to BrokerServerViewTest 

Explain why we need to do explicit checks on existences of zNodes before we create them. Address issue: https://github.com/druid-io/druid/issues/1512
This commit is contained in:
Bingkun Guo 2015-12-09 13:59:33 -06:00
parent 161c4ce288
commit 31502a920c
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
server/src/test/java/io/druid/client/BrokerServerViewTest.java
View File

@ -374,6 +374,11 @@ public class BrokerServerViewTest extends CuratorTestBase
final String zNodePathAnnounce = ZKPaths.makePath(announcementsPath, server.getHost()); final String zNodePathAnnounce = ZKPaths.makePath(announcementsPath, server.getHost());
final String zNodePathSegment = ZKPaths.makePath(inventoryPath, server.getHost()); final String zNodePathSegment = ZKPaths.makePath(inventoryPath, server.getHost());
/*
* Explicitly check whether the zNodes we are about to create exist or not,
* if exist, delete them to make sure we have a clean state on zookeeper.
* Address issue: https://github.com/druid-io/druid/issues/1512
*/
if (curator.checkExists().forPath(zNodePathAnnounce) != null) { if (curator.checkExists().forPath(zNodePathAnnounce) != null) {
curator.delete().guaranteed().forPath(zNodePathAnnounce); curator.delete().guaranteed().forPath(zNodePathAnnounce);
} }