Refactor DeclaredQuery to decouple the query definition from its introspected state
#3622
Labels
type: enhancement
A general enhancement
Milestone
Comments
This was referenced Sep 25, 2024
mp911de
added a commit
that referenced
this issue
Mar 18, 2025
…ctory` to use. Introduce QueryEnhancerSelector to EnableJpaRepositories. Also, split DeclaredQuery into two interfaces to resolve the inner cycle of query introspection while just a value object is being created. Introduce JpaQueryConfiguration to capture a multitude of configuration elements. Remove `spring.data.jpa.query.native.parser` option introduced earlier with #2989 Closes #3622 Original pull request: #3527
mp911de
added a commit
that referenced
this issue
Mar 18, 2025
Introduce refined names: EntityQuery, TemplatedQuery, ParametrizedQuery, QueryProvider. Return QueryProvider where possible. Introduce rewrite as concept on DeclaredQuery to retain its nature and track the origin of the query rewriting. Move methods solely used in tests to TestDefaultEntityQuery. Remove unused methods, fix naming, group DeclaredQuery implementations in DeclaredQueries. Add documentation. See #3622 Original pull request: #3527
mp911de
added a commit
that referenced
this issue
Mar 18, 2025
Introduce refined names: EntityQuery, TemplatedQuery, ParametrizedQuery, QueryProvider. Return QueryProvider where possible. Introduce rewrite as concept on DeclaredQuery to retain its nature and track the origin of the query rewriting. Move methods solely used in tests to TestDefaultEntityQuery. Remove unused methods, fix naming, group DeclaredQuery implementations in DeclaredQueries. Add documentation. See #3622 Original pull request: #3527
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Follow-up to #2989.
We should decouple
DeclaredQueryproperties (query string and native query flavor) from introspection properties (alias, usage of constructor expressions, …) to separate the process fromDeclaredQuerycreation and introspection so thatQueryEnhancerFactorycan be provided externally.The text was updated successfully, but these errors were encountered: