Fixed registering MemoryUsageGaugeSet

segabriel · segabriel · commit 70e3d3a91bc3 · 2018-07-31T21:09:46.000+03:00
diff --git a/benchmarks-api/src/main/java/io/scalecube/benchmarks/BenchmarksState.java b/benchmarks-api/src/main/java/io/scalecube/benchmarks/BenchmarksState.java
@@ -91,8 +91,6 @@ public final void start() {
 
     LOGGER.info("Benchmarks settings: " + settings);
 
-    settings.registry().register(settings.taskName() + "-memory", new MemoryUsageGaugeSet());
-
     if (settings.consoleReporterEnabled()) {
       consoleReporter = ConsoleReporter.forRegistry(settings.registry())
           .outputTo(System.out)
@@ -120,6 +118,8 @@ public final void start() {
       throw new IllegalStateException("BenchmarksState beforeAll() failed: " + ex, ex);
     }
 
+    settings.registry().register(settings.taskName() + "-memory", new MemoryUsageGaugeSet());
+
     if (settings.consoleReporterEnabled()) {
       consoleReporter.start(settings.reporterInterval().toMillis(), TimeUnit.MILLISECONDS);
     }

Original file line number	Diff line number	Diff line change
`@@ -91,8 +91,6 @@ public final void start() {`
`91`	`91`
`92`	`92`	`LOGGER.info("Benchmarks settings: " + settings);`
`93`	`93`
`94`		`- settings.registry().register(settings.taskName() + "-memory", new MemoryUsageGaugeSet());`
`95`		`-`
`96`	`94`	`if (settings.consoleReporterEnabled()) {`
`97`	`95`	`consoleReporter = ConsoleReporter.forRegistry(settings.registry())`
`98`	`96`	`.outputTo(System.out)`
`@@ -120,6 +118,8 @@ public final void start() {`
`120`	`118`	`throw new IllegalStateException("BenchmarksState beforeAll() failed: " + ex, ex);`
`121`	`119`	`}`
`122`	`120`
	`121`	`+ settings.registry().register(settings.taskName() + "-memory", new MemoryUsageGaugeSet());`
	`122`	`+`
`123`	`123`	`if (settings.consoleReporterEnabled()) {`
`124`	`124`	`consoleReporter.start(settings.reporterInterval().toMillis(), TimeUnit.MILLISECONDS);`
`125`	`125`	`}`