issue/DATAJDBC-430 - Allow injection of "row mapper," and "result set extractor" beans. #247
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
| One comment, I had to isolate the configurations of the beans in a separate class—the injection of other tests broken due to the rowMapper service not being available. |
uaihebert changed the title 
| Thanks for tackling this issue. I'm afraid the comment on the issue by myself:
wasn't precise enough. These attributes should be of type If you could adjust the PR according to that, I'd be happy to merge it. |
Until now, only classes without injection had support. This commit will add two new params at the `@Query` annotation. In these new params will be possible to add Spring Beans allowing the user to use Spring Beans.
uaihebert force-pushed the issue/DATAJDBC-430 branch from aa877da to 4238d73 Compare | I just finished the update. I will prepare a new PR. |
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments. Suggestions cannot be applied while the pull request is queued to merge. Suggestion cannot be applied right now. Please check back later.
Until now, only classes without injection had support. This commit will add two new params at the
@Queryannotation.In these new params will be possible to add Spring Beans allowing the user to use Spring Beans.