GROOVY-12181: Refactor async runtime and AST helpers introduced by GROOVY-9381#2725
GROOVY-12181: Refactor async runtime and AST helpers introduced by GROOVY-9381#2725daniellansun wants to merge 1 commit into
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #2725 +/- ##
==================================================
+ Coverage 69.2109% 69.2173% +0.0064%
- Complexity 34491 34514 +23
==================================================
Files 1539 1541 +2
Lines 129913 129927 +14
Branches 23669 23667 -2
==================================================
+ Hits 89914 89932 +18
+ Misses 31942 31937 -5
- Partials 8057 8058 +1
🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
⚠️ Performance Alert ⚠️
Possible performance regression was detected for benchmark.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.50.
| Benchmark suite | Current: 9d50409 | Previous: cb27031 | Ratio |
|---|---|---|---|
org.apache.groovy.bench.AckermannBench.groovy ( {"n":"7"} ) |
6.146820308109513 ms/op |
3.930829100145476 ms/op |
1.56 |
org.apache.groovy.bench.AckermannBench.java ( {"n":"5"} ) |
0.10091901247876386 ms/op |
0.031295695362762596 ms/op |
3.22 |
org.apache.groovy.bench.AckermannBench.java ( {"n":"6"} ) |
0.4588753761198549 ms/op |
0.13235591772257407 ms/op |
3.47 |
org.apache.groovy.bench.AckermannBench.java ( {"n":"7"} ) |
1.9243456060863213 ms/op |
0.5293173656330404 ms/op |
3.64 |
org.apache.groovy.bench.AryBench.groovy ( {"n":"1000"} ) |
24.675133680448354 ms/op |
15.874060997829114 ms/op |
1.55 |
org.apache.groovy.bench.AryBench.groovy ( {"n":"1000000"} ) |
26901.5370352 ms/op |
17335.853736200002 ms/op |
1.55 |
org.apache.groovy.bench.AryBench.groovyCS ( {"n":"10"} ) |
0.005777345487024153 ms/op |
0.003729125330018467 ms/op |
1.55 |
org.apache.groovy.bench.AryBench.groovyCS ( {"n":"1000"} ) |
0.13850581286904526 ms/op |
0.0370388248862454 ms/op |
3.74 |
org.apache.groovy.bench.AryBench.java ( {"n":"1000"} ) |
0.13762888248814983 ms/op |
0.04206963429286435 ms/op |
3.27 |
org.apache.groovy.bench.FiboBench.groovy ( {"n":"30"} ) |
10.600115016325447 ms/op |
6.5706982825594 ms/op |
1.61 |
org.apache.groovy.bench.FiboBench.groovy ( {"n":"31"} ) |
17.149320412900188 ms/op |
10.451395709895833 ms/op |
1.64 |
org.apache.groovy.bench.FiboBench.groovy ( {"n":"32"} ) |
28.13315974248826 ms/op |
17.604772994932308 ms/op |
1.60 |
org.apache.groovy.bench.FiboBench.groovy ( {"n":"33"} ) |
45.19156971762626 ms/op |
27.633212026182935 ms/op |
1.64 |
org.apache.groovy.bench.FiboBench.groovy ( {"n":"34"} ) |
72.77919064642857 ms/op |
44.2759107673913 ms/op |
1.64 |
org.apache.groovy.bench.DynamicDispatchColdBench.dynamicMono_java ( {"n":"20000"} ) |
920.41604 us/op |
598.55548 us/op |
1.54 |
This comment was automatically generated by workflow using github-action-benchmark.
JMH summary — classic (commit
|
| Group | Speedup | Calibrated | n |
|---|---|---|---|
| bench | 1.082 × | 0.996 × | 84 |
| core | 1.043 × | 1.008 × | 77 |
| grails | 1.210 × | 1.176 × | 80 |
Runner calibration (this run vs baseline hardware): bench 1.08× (26 rulers) · core-ag 1.10× (3 rulers) · core-hz 0.97× (3 rulers) · grails-ad 0.96× (3 rulers) · grails-ez 1.09× (3 rulers)
Baseline: dev/bench/jmh/<part>/classic/data.js on gh-pages, trailing 90 days. Daily dashboard · Per-suite raw data
JMH summary — indy (commit
|
| Group | Speedup | Calibrated | n |
|---|---|---|---|
| bench | 0.932 × | 0.944 × | 84 |
| core | 3.590 × | 3.417 × | 77 |
| grails | 2.279 × | 2.090 × | 80 |
Runner calibration (this run vs baseline hardware): bench 0.97× (26 rulers) · core-ag 1.14× (3 rulers) · core-hz 0.96× (3 rulers) · grails-ad 1.11× (3 rulers) · grails-ez 1.07× (3 rulers)
Baseline: dev/bench/jmh/<part>/indy/data.js on gh-pages, trailing 90 days. Daily dashboard · Per-suite raw data
305bc11 to
9d50409
Compare
|
✅ All tests passed ✅🏷️ Commit: 9d50409 Learn more about TestLens at testlens.app. |


https://issues.apache.org/jira/browse/GROOVY-12181