You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by vi...@apache.org on 2012/07/06 18:10:54 UTC
svn commit: r1358291 - /accumulo/trunk/test/system/auto/TestUtils.py
Author: vines
Date: Fri Jul 6 16:10:53 2012
New Revision: 1358291
URL: http://svn.apache.org/viewvc?rev=1358291&view=rev
Log:
ACCUMULO-629 - purging more logging cruft
Modified:
accumulo/trunk/test/system/auto/TestUtils.py
Modified: accumulo/trunk/test/system/auto/TestUtils.py
URL: http://svn.apache.org/viewvc/accumulo/trunk/test/system/auto/TestUtils.py?rev=1358291&r1=1358290&r2=1358291&view=diff
==============================================================================
--- accumulo/trunk/test/system/auto/TestUtils.py (original)
+++ accumulo/trunk/test/system/auto/TestUtils.py Fri Jul 6 16:10:53 2012
@@ -170,20 +170,10 @@ class TestUtilsMixin:
self.pkill(host, 'Main master$', signal=signal.SIGHUP)
self.cleanupAccumuloHandles()
- def stop_logger(self, host):
- self.pkill(host, 'Main logger$', signal=signal.SIGHUP)
- # wait for it to stop
- self.sleep(1)
- self.cleanupAccumuloHandles(0.5)
-
def start_tserver(self, host):
return self.runOn(host,
[self.accumulo_sh(), 'tserver'])
- def start_logger(self, host):
- return self.runOn(host,
- [self.accumulo_sh(), 'logger'])
-
def start_monitor(self, host):
return self.runOn(host, [self.accumulo_sh(), 'monitor'])
@@ -291,8 +281,6 @@ class TestUtilsMixin:
def start_accumulo_procs(self, safeMode=None):
self.accumuloHandles = [
- self.start_logger(host) for host in self.hosts
- ] + [
self.start_tserver(host) for host in self.hosts
] + [
self.start_monitor(self.masterHost())
@@ -388,8 +376,6 @@ class TestUtilsMixin:
[self.accumulo_sh(), 'admin', '-u', ROOT,
'-p', ROOT_PASSWORD, 'stopAll'])
self.waitForStop(handle, seconds)
- for host in self.hosts:
- self.stop_logger(host)
self.stop_monitor(self.masterHost())
self.cleanupAccumuloHandles()
# give everyone a couple seconds to completely stop