You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ma...@apache.org on 2011/12/14 22:27:34 UTC
svn commit: r1214483 -
/lucene/dev/branches/solrcloud/solr/core/src/java/org/apache/solr/cloud/Overseer.java
Author: markrmiller
Date: Wed Dec 14 21:27:34 2011
New Revision: 1214483
URL: http://svn.apache.org/viewvc?rev=1214483&view=rev
Log:
these types of exists calls do not appear to be safe
Modified:
lucene/dev/branches/solrcloud/solr/core/src/java/org/apache/solr/cloud/Overseer.java
Modified: lucene/dev/branches/solrcloud/solr/core/src/java/org/apache/solr/cloud/Overseer.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/solrcloud/solr/core/src/java/org/apache/solr/cloud/Overseer.java?rev=1214483&r1=1214482&r2=1214483&view=diff
==============================================================================
--- lucene/dev/branches/solrcloud/solr/core/src/java/org/apache/solr/cloud/Overseer.java (original)
+++ lucene/dev/branches/solrcloud/solr/core/src/java/org/apache/solr/cloud/Overseer.java Wed Dec 14 21:27:34 2011
@@ -150,9 +150,7 @@ public class Overseer implements NodeSta
synchronized (nodeStateWatches) {
if (!nodeStateWatches.containsKey(nodeName)) {
try {
- if (!zkClient.exists(path)) {
- zkClient.makePath(path);
- }
+ zkClient.makePath(path);
} catch (KeeperException e) {
if (e.code() != Code.NODEEXISTS) {
throw e;