-
Notifications
You must be signed in to change notification settings - Fork 346
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor StringBasedJdbcQuery
to no longer require BeanFactory
#1872
Comments
Not sure I understand how you envision this. The |
We ask |
schauder
added a commit
that referenced
this issue
Sep 6, 2024
The lookup is now performed by the `RowMapperFactory`. Closes #1872
mp911de
added
type: enhancement
A general enhancement
and removed
type: task
A general task
labels
Sep 16, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We already accept
RowMapperFactory
inStringBasedJdbcQuery
that could serve as abstraction to not requireBeanFactory
.JdbcQueryLookupStrategy
provides aRowMapperFactory
implementation so thatJdbcQueryLookupStrategy
could serve as entry point for aRowMapperFactory
implementation usingBeanFactory
. We can extendRowMapperFactory
to resolve bean references and reduce so coupling inStringBasedJdbcQuery
.The text was updated successfully, but these errors were encountered: