svn commit: r1148553 - in /zookeeper/trunk: CHANGES.txt src/java/test/org/apache/zookeeper/test/JMXEnv.java

[phunt]

Author: phunt
Date: Tue Jul 19 21:38:56 2011
New Revision: 1148553

URL: http://svn.apache.org/viewvc?rev=1148553&view=rev
Log:
ZOOKEEPER-1111. JMXEnv uses System.err instead of logging (Ivan Kelly via phunt)

Modified:
    zookeeper/trunk/CHANGES.txt
    zookeeper/trunk/src/java/test/org/apache/zookeeper/test/JMXEnv.java

Modified: zookeeper/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/zookeeper/trunk/CHANGES.txt?rev=1148553&r1=1148552&r2=1148553&view=diff
==============================================================================
--- zookeeper/trunk/CHANGES.txt (original)
+++ zookeeper/trunk/CHANGES.txt Tue Jul 19 21:38:56 2011
@@ -251,6 +251,9 @@ BUGFIXES: 
 
   ZOOKEEPER-1124. Multiop submitted to non-leader always fails due to timeout 
(Marshall McMullen via breed)
 
+  ZOOKEEPER-1111. JMXEnv uses System.err instead of logging
+  (Ivan Kelly via phunt)
+
 IMPROVEMENTS:
   ZOOKEEPER-724. Improve junit test integration - log harness information 
   (phunt via mahadev)

Modified: zookeeper/trunk/src/java/test/org/apache/zookeeper/test/JMXEnv.java
URL: 
http://svn.apache.org/viewvc/zookeeper/trunk/src/java/test/org/apache/zookeeper/test/JMXEnv.java?rev=1148553&r1=1148552&r2=1148553&view=diff
==============================================================================
--- zookeeper/trunk/src/java/test/org/apache/zookeeper/test/JMXEnv.java 
(original)
+++ zookeeper/trunk/src/java/test/org/apache/zookeeper/test/JMXEnv.java Tue Jul 
19 21:38:56 2011
@@ -102,10 +102,10 @@ public class JMXEnv {
         
         Set<ObjectName> found = new HashSet<ObjectName>();
         for (String name : expectedNames) {
-            System.err.println("expect:" + name);
+            LOG.info("expect:" + name);
             for (ObjectName bean : beans) {
                 if (bean.toString().contains(name)) {
-                    System.err.println("found:" + name + " " + bean);
+                    LOG.info("found:" + name + " " + bean);
                     found.add(bean);
                     break;
                 }
@@ -130,10 +130,10 @@ public class JMXEnv {
     public static Set<ObjectName> ensureOnly(String... expectedNames)
         throws IOException
     {
-        System.err.println("ensureOnly:" + Arrays.toString(expectedNames));
+        LOG.info("ensureOnly:" + Arrays.toString(expectedNames));
         Set<ObjectName> beans = ensureAll(expectedNames);
         for (ObjectName bean : beans) {
-            System.err.println("unexpected:" + bean.toString());
+            LOG.info("unexpected:" + bean.toString());
         }
         TestCase.assertEquals(0, beans.size());
         return beans;
@@ -153,7 +153,7 @@ public class JMXEnv {
         for (String name : expectedNames) {
             for (ObjectName bean : beans) {
                 if (bean.toString().contains(name)) {
-                    System.err.println("didntexpect:" + name);
+                    LOG.info("didntexpect:" + name);
                     TestCase.fail(name + " " + bean.toString());
                 }
             }
@@ -161,7 +161,7 @@ public class JMXEnv {
     }
 
     public static void dump() throws IOException {
-        System.err.println("JMXEnv.dump() follows");
+        LOG.info("JMXEnv.dump() follows");
         Set<ObjectName> beans;
         try {
             beans = conn().queryNames(
@@ -170,7 +170,7 @@ public class JMXEnv {
             throw new RuntimeException(e);
         }
         for (ObjectName bean : beans) {
-            System.err.println("bean:" + bean.toString());
+            LOG.info("bean:" + bean.toString());
         }
     }