The GitHub Actions job "CI - Groovy Joint Validation Build" on 
grails-core.git/feat/data-service-datasource-inheritance has failed.
Run started by GitHub user jamesfredley (triggered by jamesfredley).

Head commit for run:
19205e665791d6ba924e085973b92ca98979c140 / James Fredley 
<[email protected]>
fix: generate connection-aware getTransactionManager() in ServiceTransformation 
for inherited datasource services

When a @Service data service auto-inherits its datasource from the domain class 
mapping,
ServiceTransformation creates a synthetic impl class and adds 
@Transactional(connection='secondary')
to it. However, TransactionalTransform runs in SEMANTIC_ANALYSIS phase before 
the impl class
exists, so it never weaves getTransactionManager() with the correct 
connection-aware logic onto
that impl. At runtime this caused 'No Session found for current thread' because 
the default
transaction manager was used instead of the secondary one.

Fix: ServiceTransformation now directly generates the getTransactionManager() 
method on the
impl class in applyDomainConnectionToService(), mirroring exactly what 
TransactionalTransform
would have done for Service-implementing classes with a connection attribute.

Assisted-by: Claude Code <[email protected]>

Report URL: https://github.com/apache/grails-core/actions/runs/22336053469

With regards,
GitHub Actions via GitBox

Reply via email to