[borinquenkid]

What

Converts the remaining Java sources of grails-datamapping-core to Groovy: 68 .java files removed, 76 .groovy files added (90 files changed, 7604 insertions(+), 6377 deletions(-)).

Why

The maintainers asked on #16351 for the Java-to-Groovy migration to land one Gradle project at a time as a stack of PRs, bottom-up in dependency order. #16572 did grails-data-hibernate7; this PR is 10 of 30 in the next stack (base: refactor/java-to-groovy-datamapping-validation). The end state of every file is taken from the 9.0.x-java-to-groovy branch, which already carries the regression fixes found by CI.

Commits:

• 514d197 fix: keep the Java-facing API of the converted grails-datamapping-core classes
• ee34a49 refactor: convert grails-datamapping-core to Groovy

Changed outside the converted sources

• nothing

Java-facing API sweep

Every deleted .java was compared with its Groovy twin via javap -v -protected (class declaration, public/protected fields, constructors, methods incl. modifiers/throws/generics, interface default methods, nested class modifiers, and the annotations callers see). Result:

• grails-datamapping-core: 68 deleted .java, 6 difference(s) found and fixed in this PR (see the fix: commit), 0 left

See also the notes below. Remaining informational entries (widened visibility, e.g. Groovy property accessors): 13.

Verification

• classes and testClasses (plus ast/cli/testFixtures source sets where present) of the converted project(s) and of their direct dependents compile: ['grails-converters', 'grails-data-graphql-core', 'grails-data-mongodb-bson', 'grails-data-mongodb-core', 'grails-data-mongodb-ext', 'grails-data-mongodb-spring-boot', 'grails-data-neo4j-core', 'grails-data-simple', 'grails-datamapping-async', 'grails-datamapping-core-test', 'grails-datamapping-rx', 'grails-datamapping-support', 'grails-datamapping-tck', 'grails-datasource', 'grails-datastore-core', 'grails-domain-class', 'grails-events-transforms', 'grails-geb', 'grails-redis', 'grails-spring-security', 'grails-spring-security-oauth2', 'grails-testing-support-core', 'grails-testing-support-datamapping']
• No test suites were run locally (the umbrella branch was verified; CI runs them). codeStyle, rat and a whole-repo build -PskipTests are run once on the tip of the stack.

Notes

• Informational sweep entries that are not regressions: Groovy enums gain MIN_VALUE/MAX_VALUE/next()/previous(), property accessors appear for formerly package-private members, nested static classes with implicit constructors become public (they were protected), and generic type parameters that Groovy erases in constructors/methods (descriptors identical).
• Groovy gotchas that applied (see build: migrate remaining Java sources to Groovy #16351): static final without public creates a property; == calls equals(); Closure/SAM coercions; @CompileStatic with nested classes. All CI regression fixes found on the umbrella branch are included in the end state.

🤖 Generated with Claude Code

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[codecov]

Codecov Report

❌ Patch coverage is 90.36204% with 197 lines in your changes missing coverage. Please review.
✅ Project coverage is 63.5165%. Comparing base (c3fd897) to head (5b06fc3).

Files with missing lines	Patch %	Lines
...rails/datastore/gorm/jdbc/RelaxedDataBinder.groovy	71.0059%	39 Missing and 10 partials ⚠️
...-core/src/main/groovy/grails/gorm/PagedList.groovy	0.0000%	23 Missing ⚠️
...grails/datastore/gorm/finders/DynamicFinder.groovy	92.0578%	12 Missing and 10 partials ⚠️
...src/main/groovy/grails/gorm/PagedResultList.groovy	77.1930%	13 Missing ⚠️
...ils/datastore/gorm/query/AbstractResultList.groovy	83.5443%	4 Missing and 9 partials ⚠️
...rails/datastore/gorm/jdbc/DataSourceBuilder.groovy	85.5263%	6 Missing and 5 partials ⚠️
...tore/gorm/transform/AstPropertyResolveUtils.groovy	84.8485%	4 Missing and 6 partials ⚠️
...nnections/DataSourceConnectionSourceFactory.groovy	77.5000%	9 Missing ⚠️
...che/grails/datamapping/aot/GormRuntimeHints.groovy	64.2857%	4 Missing and 1 partial ⚠️
...s/datastore/gorm/events/DomainEventListener.groovy	95.8333%	1 Missing and 4 partials ⚠️
... and 17 more

Additional details and impacted files

@@                                   Coverage Diff                                    @@
##             refactor/java-to-groovy-datamapping-validation     #16588        +/-   ##
========================================================================================
- Coverage                                           63.5473%   63.5165%   -0.0308%     
- Complexity                                            30035      30384       +349     
========================================================================================
  Files                                                  2243       2243                
  Lines                                                110066     109858       -208     
  Branches                                              20246      20269        +23     
========================================================================================
- Hits                                                  69944      69778       -166     
+ Misses                                                30866      30737       -129     
- Partials                                               9256       9343        +87     

Files with missing lines	Coverage Δ
.../gorm/bootstrap/support/InstanceFactoryBean.groovy	100.0000% <100.0000%> (ø)
...astore/gorm/finders/DynamicFinderInvocation.groovy	100.0000% <100.0000%> (ø)
...grails/datastore/gorm/finders/FinderSupport.groovy	80.0000% <100.0000%> (ø)
...ils/datastore/gorm/finders/ListResultFinder.groovy	100.0000% <100.0000%> (ø)
...g/grails/datastore/gorm/jdbc/DatabaseDriver.groovy	85.1852% <100.0000%> (ø)
...ons/CachedDataSourceConnectionSourceFactory.groovy	100.0000% <100.0000%> (ø)
...jdbc/connections/DataSourceConnectionSource.groovy	100.0000% <100.0000%> (ø)
...transform/DetachedCriteriaASTTransformation.groovy	100.0000% <100.0000%> (ø)
...query/transform/DetachedCriteriaTransformer.groovy	79.4335% <ø> (ø)
.../query/transform/GormQuerySafetyTransformer.groovy	83.6667% <ø> (ø)
... and 31 more

... and 9 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[borinquenkid]

SUPERSEDED: the skipTests/skipCodeStyle flag described below was removed. See the newer comment about the CHANGEME FOR ACCEPTANCE workflow-trigger commit, which is the only temporary CI change on this branch.

Temporary CI flag on this stacked branch (BLOCKER before merge).

This Java-to-Groovy chain is a stack of PRs, and every push to any of them starts the full CI matrix, which is more runner time than the ASF allows us. Every PR in the stack except the one closest to its release branch (#16576) therefore carries one temporary commit, CHANGEME FOR ACCEPTANCE, that adds this to the root gradle.properties:

skipTests=true
skipCodeStyle=true

With it, CI on this PR compiles but does not run tests or code style. The full build runs on #16576 (closest to 10.0.x), and each PR gets it when the PR below it has merged.

Condition for merging this PR: that gradle.properties block (and the CHANGEME FOR ACCEPTANCE commit) must be dropped from this branch first, and CI must then be green with the tests running. A green check on this PR as it stands does not mean the tests ran. Rollout of the commit is branch by branch, so a branch may not carry it yet.

[borinquenkid]

Update: this branch now carries a different temporary CI commit (BLOCKER before merge).

The earlier skipTests/skipCodeStyle flag in gradle.properties did not help enough: the functional jobs still compile the whole repository (about 57 minutes each) on every push, so it only saved around a fifth of the time. It has been removed.

In its place, every PR in this stack except #16576 (the one closest to 10.0.x) carries one temporary commit, CHANGEME FOR ACCEPTANCE: disable PR workflow triggers. It changes the pull_request: trigger of each workflow to branches: ['CHANGEME-FOR-ACCEPTANCE'], a branch that does not exist, so pushing, rebasing or force-pushing this branch starts no jobs.

Condition for merging this PR: that commit must be reverted on this branch first (git revert of the CHANGEME FOR ACCEPTANCE commit), and CI must then run and be green. The absence of CI checks on this PR is deliberate and does not mean the code was tested. The full build runs on #16576, and each PR gets it when the PR below it has merged.

[testlens-app]

🔎 No tests executed 🔎

🏷️ Commit: 9258cfb
▶️ Tests: 0 executed
⚪️ Checks: 42/42 completed

---

Learn more about TestLens at testlens.app/docs.