The GitHub Actions job "CI - Groovy Joint Validation Build" on grails-core.git/feat/gorm-registry-consolidated has failed. Run started by GitHub user borinquenkid (triggered by borinquenkid).
Head commit for run: 1b028adfbe935d7268b9f57fd5b93dd79cbb51b8 / Walter Duque de Estrada <[email protected]> fix: address findings from adversarial review of the consolidated branch - DatastoreUtils.execute/doWithSession: stack via bindNewSession and release via unbindSession so a bound-but-empty SessionHolder can no longer fail a later bind with 'already value bound' (spec added) - TransactionSynchronizationSessionResolver.resolve(): resolve down the stack past stale sessions and unbind a holder emptied by eviction unless a transaction owns it, so eviction can never poison later binds (specs added, including the transaction-owned case) - GormStaticApi.withStatelessSession: restore connectStateless() and the UnsupportedOperationException guard; it had been rewired through executeWithNewSession and silently handed out stateful sessions - GormStaticApi.saveAll: drop the mid-transaction flush the 8.0.x baseline never had; the spec now exercises the documented session-scoped contract - TenantDelegatingGormOperations.delete(instance, params): delegate to delete, not save - pre-existing data-integrity bug in a touched class (spec added) - GormApiResolver selectors: read the current tenant per-datastore instead of the no-arg CurrentTenantHolder.get(), which returned an arbitrary datastore's tenant and could route one datastore's entities by another's tenant id - GormRegistry.registerEntityDatastores: publish the rebuilt routing map atomically instead of remove-then-repopulate - AbstractGormApiRegistry.getDirect: re-validate a freshly cached qualified API against the current default API and retract it if superseded by register() - GormRegistry.removeDatastore: clear normalization caches once the last datastore is gone (classloader retention, per-tenant-id growth) - Document the ActiveSessionDatastoreSelector 10-datastore fallback bound - AbstractDatastore.setApplicationContext: drop the dead instanceof check - Normalize import grouping in the new datastore-core specs Co-Authored-By: Claude Fable 5 <[email protected]> Report URL: https://github.com/apache/grails-core/actions/runs/30477802119 With regards, GitHub Actions via GitBox
