accumulo-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From els...@apache.org
Subject [08/16] git commit: ACCUMULO-2334 Use the ZooReaderWriter exists() and make a slightly better comment.
Date Tue, 11 Feb 2014 18:41:49 GMT
ACCUMULO-2334 Use the ZooReaderWriter exists() and make a slightly better comment.


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/fafa4296
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/fafa4296
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/fafa4296

Branch: refs/heads/1.5.1-SNAPSHOT
Commit: fafa42961be8c6f457c5519852b398c488654980
Parents: fd31595
Author: Josh Elser <elserj@apache.org>
Authored: Tue Feb 11 12:05:01 2014 -0500
Committer: Josh Elser <elserj@apache.org>
Committed: Tue Feb 11 12:05:01 2014 -0500

----------------------------------------------------------------------
 .../org/apache/accumulo/server/watcher/MonitorLog4jWatcher.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/fafa4296/server/src/main/java/org/apache/accumulo/server/watcher/MonitorLog4jWatcher.java
----------------------------------------------------------------------
diff --git a/server/src/main/java/org/apache/accumulo/server/watcher/MonitorLog4jWatcher.java
b/server/src/main/java/org/apache/accumulo/server/watcher/MonitorLog4jWatcher.java
index c0cef08..d0ca27f 100644
--- a/server/src/main/java/org/apache/accumulo/server/watcher/MonitorLog4jWatcher.java
+++ b/server/src/main/java/org/apache/accumulo/server/watcher/MonitorLog4jWatcher.java
@@ -60,7 +60,7 @@ public class MonitorLog4jWatcher extends FileWatchdog implements Watcher
{
   public void run() {
     try {
       // Initially set the logger if the Monitor's log4j advertisement node exists
-      if (ZooReaderWriter.getInstance().getZooKeeper().exists(path, this) != null)
+      if (ZooReaderWriter.getInstance().exists(path, this))
         updateMonitorLog4jLocation();
       log.info("Set watch for Monitor Log4j watcher");
     } catch (Exception e) {
@@ -73,7 +73,7 @@ public class MonitorLog4jWatcher extends FileWatchdog implements Watcher
{
   @Override
   protected void doOnChange() {
     // this method gets called in the parent class' constructor
-    // I'm not sure of a better way to get around this. final helps though.
+    // I'm not sure of a better way to get around this. The final modifier helps though.
     if (null == lock) {
       resetLogger();
       return;


Mime
View raw message