You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by bv...@apache.org on 2012/11/30 21:26:37 UTC
svn commit: r1415838 - in
/camel/trunk/components/camel-zookeeper/src/main/java/org/apache/camel/component/zookeeper:
ZooKeeperConfiguration.java ZooKeeperEndpoint.java
Author: bvahdat
Date: Fri Nov 30 20:26:36 2012
New Revision: 1415838
URL: http://svn.apache.org/viewvc?rev=1415838&view=rev
Log:
CAMEL-5838: The 'awaitExistence' property's setter/getter method names should match with the field's name itself. As well did some tiny polishment.
Modified:
camel/trunk/components/camel-zookeeper/src/main/java/org/apache/camel/component/zookeeper/ZooKeeperConfiguration.java
camel/trunk/components/camel-zookeeper/src/main/java/org/apache/camel/component/zookeeper/ZooKeeperEndpoint.java
Modified: camel/trunk/components/camel-zookeeper/src/main/java/org/apache/camel/component/zookeeper/ZooKeeperConfiguration.java
URL: http://svn.apache.org/viewvc/camel/trunk/components/camel-zookeeper/src/main/java/org/apache/camel/component/zookeeper/ZooKeeperConfiguration.java?rev=1415838&r1=1415837&r2=1415838&view=diff
==============================================================================
--- camel/trunk/components/camel-zookeeper/src/main/java/org/apache/camel/component/zookeeper/ZooKeeperConfiguration.java (original)
+++ camel/trunk/components/camel-zookeeper/src/main/java/org/apache/camel/component/zookeeper/ZooKeeperConfiguration.java Fri Nov 30 20:26:36 2012
@@ -37,7 +37,7 @@ public class ZooKeeperConfiguration impl
private int sessionId;
private byte[] password;
private String path;
- private boolean awaitCreation = true;
+ private boolean awaitExistence = true;
private boolean repeat;
private boolean listChildren;
private boolean shouldCreate;
@@ -86,7 +86,6 @@ public class ZooKeeperConfiguration impl
}
public String getConnectString() {
- // return StringHelper.toStringCommaSeparated(servers);
StringBuilder b = new StringBuilder();
for (String server : servers) {
b.append(server).append(",");
@@ -129,11 +128,11 @@ public class ZooKeeperConfiguration impl
}
public boolean shouldAwaitExistence() {
- return awaitCreation;
+ return awaitExistence;
}
- public void setAwaitExistance(boolean awaitCreation) {
- this.awaitCreation = awaitCreation;
+ public void setAwaitExistance(boolean awaitExistence) {
+ this.awaitExistence = awaitExistence;
}
public long getBackoff() {
Modified: camel/trunk/components/camel-zookeeper/src/main/java/org/apache/camel/component/zookeeper/ZooKeeperEndpoint.java
URL: http://svn.apache.org/viewvc/camel/trunk/components/camel-zookeeper/src/main/java/org/apache/camel/component/zookeeper/ZooKeeperEndpoint.java?rev=1415838&r1=1415837&r2=1415838&view=diff
==============================================================================
--- camel/trunk/components/camel-zookeeper/src/main/java/org/apache/camel/component/zookeeper/ZooKeeperEndpoint.java (original)
+++ camel/trunk/components/camel-zookeeper/src/main/java/org/apache/camel/component/zookeeper/ZooKeeperEndpoint.java Fri Nov 30 20:26:36 2012
@@ -114,11 +114,6 @@ public class ZooKeeperEndpoint extends D
getConfiguration().setServers(servers);
}
- // TODO: check how to allow should to indicate getter of an attribute pair
-// public boolean shouldListChildren() {
-// return getConfiguration().listChildren();
-// }
-
@ManagedAttribute
public boolean getListChildren() {
return getConfiguration().listChildren();
@@ -129,10 +124,6 @@ public class ZooKeeperEndpoint extends D
getConfiguration().setListChildren(listChildren);
}
-// public boolean shouldCreate() {
-// return getConfiguration().shouldCreate();
-// }
-
@ManagedAttribute
public boolean getCreate() {
return getConfiguration().shouldCreate();
@@ -153,10 +144,6 @@ public class ZooKeeperEndpoint extends D
getConfiguration().setBackoff(backoff);
}
-// public boolean shouldAwaitExistence() {
-// return getConfiguration().shouldAwaitExistence();
-// }
-
@ManagedAttribute
public boolean getAwaitExistence() {
return getConfiguration().shouldAwaitExistence();