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
Implement @RegisterCollector with tests and short doc #2377
Conversation
9c8e13c
to
3509081
Compare
3509081
to
c54e62e
Compare
thank you so much, have had no time lately my only thought that I was hopefully gonna get around to - would it be better to use a CollectorSupplier kind of class in the annotation? |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
@@ -4636,7 +4636,14 @@ the `User.id` and `User.name` fields. Likewise for `r_id` and `r_name` into the | |||
The link:{jdbidocs}/sqlobject/config/RegisterConstructorMapper.html[@RegisterConstructorMapper^] annotation may be repeated multiple times on | |||
the same type or method to register multiple constructor mappers. | |||
|
|||
==== @RegisterCollectorFactory |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
RegisterCollector ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this would be a better fit after @UseRowReducer
. The docs are so bare bones that, without an example, this will probably not find any users.
I will merge this and then fix the docs. |
I took the PR and hopefully got it over the finish line with a test and some simple docs.
Bonus fix: try to use the Collector's declared element type, so that you may collect into things that aren't
Iterable
s.Fixes #2357