Skip to content
This repository was archived by the owner on Mar 13, 2025. It is now read-only.

Conversation

@puneetbehl
Copy link
Contributor

@puneetbehl puneetbehl commented May 26, 2020

to create bean DSL for data services.

Fixes #202

to create bean DSL for data services.
* Removed hibernateDatastoreServiceRegistry bean as that was redundant.
* Use SoftServiceLoader to prevent instantiating the service.
@puneetbehl puneetbehl requested a review from jameskleeh May 28, 2020 19:25
logger configuration to remove debug logs.
@puneetbehl
Copy link
Contributor Author

puneetbehl commented May 28, 2020

Please merge apache/grails-data-mapping#1350 before this pull request

@jameskleeh
Copy link
Contributor

Get with @jeffbrown on the use cases that he ran into to ensure we're covering the bases

* Update the `if()` check to filter out some unrelated services.
* Rename GormMethodInvokingFactoryBean -> DatastoreServiceMethodInvokingFactoryBean
to verify that the autowire by name works for the data service.
* Use loadServices method from the base class.
Use `def` to verify autowire by name.
* to verify that regular Grails service correctly autowire by type in the Data service.
@jameskleeh jameskleeh merged commit 30f8cda into 7.0.x Jun 9, 2020
@puneetbehl puneetbehl added this to the 7.0.4 milestone Jul 12, 2020
@puneetbehl puneetbehl deleted the 11207 branch July 22, 2021 05:30
jamesfredley pushed a commit that referenced this pull request Feb 19, 2025
…ersion

Update dependency org.liquibase:liquibase-core to v4 - manual
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants