RFR: 7903596: JMH: Explicitly enable annotation processors [v3]
Jaikiran Pai
jpai at openjdk.org
Fri Dec 6 02:08:31 UTC 2024
> Can I please get a review of this change which proposes to address the issue noted in https://bugs.openjdk.org/browse/CODETOOLS-7903596?
>
> As noted in that issue, when a benchmark generated using Java 22 is run, at runtime it fails with:
>
>
> java -jar target/benchmarks.jar
> Exception in thread "main" java.lang.RuntimeException: ERROR: Unable to find the resource: /META-INF/BenchmarkList
> at org.openjdk.jmh.runner.AbstractResourceReader.getReaders(AbstractResourceReader.java:98)
> at org.openjdk.jmh.runner.BenchmarkList.find(BenchmarkList.java:124)
> at org.openjdk.jmh.runner.Runner.internalRun(Runner.java:252)
> at org.openjdk.jmh.runner.Runner.run(Runner.java:208)
> at org.openjdk.jmh.Main.main(Main.java:71)
>
>
> This is due to annotation processors being disabled by default during compilation starting Java 22. The commit in this PR, explicitly enables the `org.openjdk.jmh.generators.BenchmarkProcessor` annotation processor to allow for it to generate the necessary benchmark resources.
>
> I've tested this change locally, with Java 8, Java 21 as well as Java 22 and the newly generated benchmark is now functional on all these versions.
Jaikiran Pai has updated the pull request incrementally with one additional commit since the last revision:
Contributed-by Aleksey: Explicitly apply the annotation processor
-------------
Changes:
- all: https://git.openjdk.org/jmh/pull/125/files
- new: https://git.openjdk.org/jmh/pull/125/files/29c7950c..2c402960
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jmh&pr=125&range=02
- incr: https://webrevs.openjdk.org/?repo=jmh&pr=125&range=01-02
Stats: 54 lines in 4 files changed: 25 ins; 28 del; 1 mod
Patch: https://git.openjdk.org/jmh/pull/125.diff
Fetch: git fetch https://git.openjdk.org/jmh.git pull/125/head:pull/125
PR: https://git.openjdk.org/jmh/pull/125
More information about the jmh-dev
mailing list