jdk17u-dev icon indicating copy to clipboard operation
jdk17u-dev copied to clipboard

8299234: JMX Repository.query performance

Open alexeybakhtin opened this issue 2 years ago • 10 comments
trafficstars

Clean backport of JDK-8299234 to JDK17u

Performance of the original code: JmxBenchmark.queryMBeansForMyObjectName 1000 thrpt 10 84.794 ± 0.784 ops/s JmxBenchmark.queryMBeansForObjectName 1000 thrpt 10 145.286 ± 0.798 ops/s JmxBenchmark.queryNamesForMyObjectName 1000 thrpt 10 80.715 ± 2.290 ops/s JmxBenchmark.queryNamesForObjectName 1000 thrpt 10 147.780 ± 4.246 ops/s

Performance after patch applied: JmxBenchmark.queryMBeansForMyObjectName 1000 thrpt 10 106.338 ± 4.865 ops/s JmxBenchmark.queryMBeansForObjectName 1000 thrpt 10 216.965 ± 3.543 ops/s JmxBenchmark.queryNamesForMyObjectName 1000 thrpt 10 102.794 ± 4.874 ops/s JmxBenchmark.queryNamesForObjectName 1000 thrpt 10 236.722 ± 1.957 ops/s


Progress

  • [x] Change must not contain extraneous whitespace
  • [x] Commit message must refer to an issue
  • [ ] JDK-8299234 needs maintainer approval

Issue

  • JDK-8299234: JMX Repository.query performance (Enhancement - P4)

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk17u-dev.git pull/1182/head:pull/1182
$ git checkout pull/1182

Update a local copy of the PR:
$ git checkout pull/1182
$ git pull https://git.openjdk.org/jdk17u-dev.git pull/1182/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 1182

View PR using the GUI difftool:
$ git pr show -t 1182

Using diff file

Download this PR as a diff file:
https://git.openjdk.org/jdk17u-dev/pull/1182.diff

Webrev

Link to Webrev Comment

alexeybakhtin avatar Feb 21 '23 10:02 alexeybakhtin