Use @teardown(Level.Invocation) instead of Iteration

In a "normal" JMH benchmark there are many invocations per iteration but by default there is only one. We do need to kill the JVM after every launch though, so, while it doesn't make any difference for the default configuration, it's safer to use Level.Invocation. See gh-7
dsyer · Apr 27, 2018 · 55ee5d2 · 55ee5d2
1 parent c028d1c
commit 55ee5d2
Show file tree

Hide file tree

Showing 7 changed files with 8 additions and 8 deletions.
diff --git a/flux/src/main/java/com/example/bench/MainBenchmark.java b/flux/src/main/java/com/example/bench/MainBenchmark.java
@@ -71,7 +71,7 @@ public MainState() {
 			super("target", "--server.port=0");
 		}
 
-		@TearDown(Level.Iteration)
+		@TearDown(Level.Invocation)
 		public void stop() throws Exception {
 			super.after();
 		}

diff --git a/flux/src/main/java/com/example/bench/MiniBenchmark.java b/flux/src/main/java/com/example/bench/MiniBenchmark.java
@@ -88,7 +88,7 @@ public MainState() {
 			super("target");
 		}
 
-		@TearDown(Level.Iteration)
+		@TearDown(Level.Invocation)
 		public void stop() throws Exception {
 			super.after();
 		}

diff --git a/static/src/main/java/com/example/bench/CdsBenchmark.java b/static/src/main/java/com/example/bench/CdsBenchmark.java
@@ -120,7 +120,7 @@ protected void customize(List<String> args) {
 			super.customize(args);
 		}
 
-		@TearDown(Level.Iteration)
+		@TearDown(Level.Invocation)
 		public void stop() throws Exception {
 			super.after();
 		}

diff --git a/static/src/main/java/com/example/bench/MainBenchmark.java b/static/src/main/java/com/example/bench/MainBenchmark.java
@@ -81,7 +81,7 @@ public MainState() {
 			super("target", "--server.port=0");
 		}
 
-		@TearDown(Level.Iteration)
+		@TearDown(Level.Invocation)
 		public void stop() throws Exception {
 			super.after();
 		}

diff --git a/static/src/main/java/com/example/bench/OldBenchmark.java b/static/src/main/java/com/example/bench/OldBenchmark.java
@@ -80,7 +80,7 @@ public OldState() {
 			super("target", "--server.port=0");
 		}
 
-		@TearDown(Level.Iteration)
+		@TearDown(Level.Invocation)
 		public void stop() throws Exception {
 			super.after();
 		}

diff --git a/static/src/main/java/com/example/bench/SnapBenchmark.java b/static/src/main/java/com/example/bench/SnapBenchmark.java
@@ -77,7 +77,7 @@ public ApplicationState() {
 			super("target", "--server.port=0");
 		}
 
-		@TearDown(Level.Iteration)
+		@TearDown(Level.Invocation)
 		public void stop() throws Exception {
 			super.after();
 		}
@@ -101,7 +101,7 @@ public EndpointState() {
 			super("target", "--server.port=0", "--endpoints.default.web.enabled=true");
 		}
 
-		@TearDown(Level.Iteration)
+		@TearDown(Level.Invocation)
 		public void stop() throws Exception {
 			super.after();
 		}

diff --git a/static/src/main/java/com/example/bench/StripBenchmark.java b/static/src/main/java/com/example/bench/StripBenchmark.java
@@ -70,7 +70,7 @@ public ApplicationState() {
 			super("target", "--server.port=0");
 		}
 
-		@TearDown(Level.Iteration)
+		@TearDown(Level.Invocation)
 		public void stop() throws Exception {
 			super.after();
 		}