You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@samza.apache.org by na...@apache.org on 2017/04/07 22:17:14 UTC

samza git commit: SAMZA-1194 - Commenting out testJmxReporter flaky test

Repository: samza
Updated Branches:
  refs/heads/master fb51bfae2 -> c74722b47


SAMZA-1194 - Commenting out testJmxReporter flaky test

TestJmxReporter brings up an JmxServer and uses it to test the reporter behavior. However, due to failures in bringing up/connecting to JMX server, this fails intermittently. Temporarily disabling it.

Author: Shanthoosh Venkataraman <sv...@linkedin.com>

Reviewers: Navina Ramesh <na...@apache.org>

Closes #118 from shanthoosh/disable_jmx_reporter_tests


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

Branch: refs/heads/master
Commit: c74722b4709962149a2989d66dbc51a68eb469e6
Parents: fb51bfa
Author: Shanthoosh Venkataraman <sv...@linkedin.com>
Authored: Fri Apr 7 15:17:10 2017 -0700
Committer: nramesh <nr...@linkedin.com>
Committed: Fri Apr 7 15:17:10 2017 -0700

----------------------------------------------------------------------
 .../scala/org/apache/samza/metrics/reporter/TestJmxReporter.scala | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/samza/blob/c74722b4/samza-core/src/test/scala/org/apache/samza/metrics/reporter/TestJmxReporter.scala
----------------------------------------------------------------------
diff --git a/samza-core/src/test/scala/org/apache/samza/metrics/reporter/TestJmxReporter.scala b/samza-core/src/test/scala/org/apache/samza/metrics/reporter/TestJmxReporter.scala
index 88370dc..431e14c 100644
--- a/samza-core/src/test/scala/org/apache/samza/metrics/reporter/TestJmxReporter.scala
+++ b/samza-core/src/test/scala/org/apache/samza/metrics/reporter/TestJmxReporter.scala
@@ -63,7 +63,8 @@ object TestJmxReporter {
 class TestJmxReporter {
   import TestJmxReporter.url
 
-  @Test
+  // TODO: Fix in SAMZA-1196
+  //@Test
   def testJmxReporter {
     val registry = new MetricsRegistryMap
     val jvm = new JvmMetrics(registry)