[
https://issues.apache.org/jira/browse/GROOVY-12392?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18113585#comment-18113585
]
ASF GitHub Bot commented on GROOVY-12392:
-----------------------------------------
github-actions[bot] commented on PR #2915:
URL: https://github.com/apache/groovy/pull/2915#issuecomment-5610797514
### JMH summary — indy (commit `ee596af`)
Speedup vs trailing 90-day baseline on gh-pages. Higher = faster.
`1.00` = in line with history. Per-benchmark ratio, geomean within group.
Time-per-op units inverted so direction is consistent. The *calibrated*
column divides out this runner's speed vs the baseline hardware, as
measured by Groovy-independent pure-Java ruler benchmarks.
| Group | Speedup | Calibrated | n |
|--------|---------|------------|---|
| bench | 1.061 × | 1.126 × | 124 |
| core | 4.225 × | 4.054 × | 108 |
| grails | 2.920 × | 2.897 × | 80 |
No benchmark is ≥1.5× slower than its 90-day baseline.
<sub>Runner calibration (this run vs baseline hardware): bench 0.96× (27
rulers) · core-ag 1.11× (3 rulers) · core-hz 0.93× (3 rulers) · grails-ad 1.11×
(3 rulers) · grails-ez 0.93× (3 rulers)</sub>
<sub>Baseline: <code>dev/bench/jmh/<part>/indy/data.js</code> on
gh-pages, trailing 90 days. <a
href="https://apache.github.io/groovy/dev/bench/jmh/summary.html";>Daily
dashboard</a> · <a
href="https://apache.github.io/groovy/dev/bench/jmh/";>Per-suite raw
data</a></sub>
<!
> NativeImageMetadataGenerator references deprecated-for-removal Java9
> --------------------------------------------------------------------
>
> Key: GROOVY-12392
> URL: https://issues.apache.org/jira/browse/GROOVY-12392
> Project: Groovy
> Issue Type: Bug
> Reporter: Daniel Sun
> Assignee: Daniel Sun
> Priority: Major
>
> h3. Context
> GROOVY-11906 deprecated {{org.codehaus.groovy.vmplugin.v9.Java9}} with
> {{forRemoval = true}} and made {{Java17}} the only {{VMPluginFactory}} entry
> point. GROOVY-12365 later added {{NativeImageMetadataGenerator}}, which still
> called {{Java9.class.getName()}} to register default-import class lookups
> ({{groovy.lang.GroovySystem}} and {{groovy.beans.ListenerList}}).
> Those names are loaded through {{GroovyClassLoader.loadClass}}, so they are
> not constant {{Class.forName}} arguments GraalVM can fold. The generator must
> emit {{typeReached}} metadata for them.
> h3. Problem
> {{compileJava}} with {{-Xlint:deprecation}} reports two {{removal}} warnings,
> which GitHub Actions surfaces as annotations:
> {noformat}
> NativeImageMetadataGenerator.java: warning: [removal] Java9 in
> org.codehaus.groovy.vmplugin.v9 has been deprecated and marked for removal
> addName(Java9.class.getName(), GroovySystem.class.getName(), NONE);
> NativeImageMetadataGenerator.java: warning: [removal] Java9 in
> org.codehaus.groovy.vmplugin.v9 has been deprecated and marked for removal
> addName(Java9.class.getName(), "groovy.beans.ListenerList", NONE);
> {noformat}
> Separately, {{Java9.checkAccessible}} looks up JDK 8 packages that are now
> concealed or exported-but-not-open with
> {{ConcurrentHashMap.computeIfAbsent}}. Callers only test {{contains()}}, so a
> miss inserts an empty mutable {{HashSet}} into a map that is otherwise filled
> once at class init.
> h3. Expected
> * Building groovy-core does not emit {{removal}} warnings for {{Java9}}.
> * Default-import reachability metadata is conditional on {{Java17}}, the
> plugin Groovy 6 actually constructs, and is generated as its own family
> rather than as a rider on invokedynamic metadata.
> * Unknown module names are not inserted into the package-list maps.
> h3. Proposed change
> * Register the two lookups under {{Java17.class.getName()}} in a dedicated
> {{defaultImportClassLookups()}} step of {{generate()}}.
> * Replace the two {{computeIfAbsent}} helpers with
> {{getOrDefault(module.getName(), Set.of())}}.
> h3. Related
> GROOVY-11906, GROOVY-12365
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
