update jmh plugin settings (#410)

* update jmh plugin settings

exclude test dependencies from jmh jar
fix duplicate jmh generated classes (jmh-generated-classes, jmh-generated-sources)
fix broken jmh metadata files (BenchmarkList, CompilerHints)
fix jmh command line jar (inline dependencies with shadow plugin)
fix jmh task for root scope (stop generate dummy jar)
cleanup redundant out directory generated by IDEA
remove slf4j and log4j dependencies (replace with slf4j-nop logger)
add visual jmh report

* fix build

Author: qweek

build.gradle

@@ -15,11 +15,15 @@
  */
 
 plugins {
-    id "com.gradle.build-scan" version "1.9"
-    id 'com.jfrog.bintray' version '1.7.3'
+    id 'com.gradle.build-scan' version '1.9' // declare before any other plugin
+
     id 'com.github.sherter.google-java-format' version '0.6'
+    id 'com.github.johnrengelman.shadow' version '2.0.1' apply false
+    id 'me.champeau.gradle.jmh' version '0.4.4' apply false
+    id 'io.morethan.jmhreport' version '0.6.2.1' apply false
+
     id 'com.jfrog.artifactory' version '4.5.2'
-    id 'me.champeau.gradle.jmh' version '0.4.4'
+    id 'com.jfrog.bintray' version '1.7.3'
 }
 
 repositories {

gradle/wrapper/gradle-wrapper.jar

@@ -14,16 +14,56 @@
  * limitations under the License.
  */
 
+apply plugin: 'com.github.johnrengelman.shadow'
 apply plugin: 'me.champeau.gradle.jmh'
+apply plugin: 'io.morethan.jmhreport'
+
+// disable tasks to stop generating duplicate files
+jmhClasses.enabled = false
+jmhRunBytecodeGenerator.enabled = false
+jmhCompileGeneratedClasses.enabled = false
+
+jmhJar {
+    // add jmh classes to shadow jar
+    from project.configurations.jmh
+    from project.configurations.jmhRuntime
+
+    // exclude logging classes
+    exclude 'org.slf4j:slf4j-log4j12'
+    exclude 'log4j:log4j'
+}
 
 jmh {
-    jmhVersion = "1.19"
-    profilers = ['gc']
-    zip64 = true
+    jmhVersion = '1.19'
+    includeTests = false
     duplicateClassesStrategy = DuplicatesStrategy.WARN
+    zip64 = true
+
+    jvmArgs = ['-XX:+UnlockCommercialFeatures', '-XX:+FlightRecorder']
+    // NOTE: uncomment to add specific options
+    // jvmArgsAppend = ['-XX:+UseG1GC', '-Xms4g', '-Xmx4g']
+    profilers = ['gc']
+    resultFormat = 'JSON'
+
+    // include = ['io.rsocket.RSocketPerf.fireAndForgetHello']
 }
 
-dependencies {
-    jmh "org.openjdk.jmh:jmh-core:1.19"
-    jmh "org.openjdk.jmh:jmh-generator-annprocess:1.19"
+// run report generation after benchmark
+tasks.jmh.finalizedBy tasks.jmhReport
+
+jmhReport {
+    jmhResultPath = project.file('build/reports/jmh/results.json')
+    jmhReportOutput = project.file('build/reports/jmh')
+}
+
+// remove directory generated by IDEA during benchmark
+clean {
+    project.delete('out')
 }
+
+dependencies {
+    jmh "org.openjdk.jmh:jmh-core:${jmh.jmhVersion}"
+    jmh "org.openjdk.jmh:jmh-generator-annprocess:${jmh.jmhVersion}"
+
+    jmhRuntime "org.slf4j:slf4j-nop:$slf4jVersion"
+}

rsocket-core/build.gradle

@@ -19,9 +19,8 @@
 
 @BenchmarkMode(Mode.Throughput)
 @Fork(
-  value = 1,
-  jvmArgsAppend = {"-XX:+UnlockCommercialFeatures", "-XX:+FlightRecorder"}
-) // , "-Dio.netty.leakDetection.level=advanced"})
+  value = 1 // , jvmArgsAppend = {"-Dio.netty.leakDetection.level=advanced"}
+)
 @Warmup(iterations = 10)
 @Measurement(iterations = 10_000)
 @State(Scope.Thread)

rsocket-core/src/jmh/java/io/rsocket/FragmentationPerf.java

@@ -41,8 +41,7 @@
 
 @BenchmarkMode(Mode.Throughput)
 @Fork(
-  value = 1,
-  jvmArgsAppend = {"-XX:+UnlockCommercialFeatures", "-XX:+FlightRecorder"}
+  value = 1 // , jvmArgsAppend = {"-Dio.netty.leakDetection.level=advanced"}
 )
 @Warmup(iterations = 10)
 @Measurement(iterations = 10)

rsocket-core/src/jmh/java/io/rsocket/RSocketPerf.java

@@ -14,25 +14,11 @@
  * limitations under the License.
  */
 
-apply plugin: 'me.champeau.gradle.jmh'
-
-jmh {
-    jmhVersion = "1.19"
-    jvmArgs = ['-XX:+UseG1GC', '-Xmx4g', '-Xms4g', '-XX:+UnlockCommercialFeatures', '-XX:+FlightRecorder']
-    profilers = ['gc']
-    zip64 = true
-    warmupBatchSize = 10
-    iterations = 500
-    duplicateClassesStrategy = DuplicatesStrategy.WARN
-}
-
 dependencies {
     compile project(':rsocket-core')
     compile project(':rsocket-spectator')
     compile project(':rsocket-transport-netty')
     compile project(':rsocket-transport-local')
 
-    jmh "org.openjdk.jmh:jmh-generator-annprocess:1.19"
-
     testCompile project(':rsocket-test')
 }