DATAREDIS-503 - Add HashMapper implementation based on MappingRedisConverter. #194
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
…nverter. Prepare issue branch.
…nverter. We now support mapping of simple and complex types to Redis HASH structures applying the same structure as the Repository support. This allows Object to hash Mapping and its direct usage via RedisTemplate without the need of explicitly having to use the repository abstraction.
mp911de
added a commit
that referenced
this pull request
Apr 29, 2016
Rename ConvertingHashMapper to ObjectHashMapper. Use generic type parameter in ObjectHashMapper. Add JavaDoc documentation to HashMapper. Add hash mapping to reference documentation. Original pull request: #194.
|
@mp911de I left the generic type out by intention as it suggests the mapper is only supposed to to be usable with one specific type which is not the case. The returned type is actually defined via |
mp911de
force-pushed
the
issue/DATAREDIS-503
branch
from
d1c7548 to
8ef926f
Compare
mp911de
added a commit
that referenced
this pull request
Apr 30, 2016
Rename ConvertingHashMapper to ObjectHashMapper. Use generic type parameter in ObjectHashMapper. Add JavaDoc documentation to HashMapper. Add hash mapping to reference documentation. Original pull request: #194.
Rename ConvertingHashMapper to ObjectHashMapper. Add JavaDoc documentation to HashMapper. Add hash mapping to reference documentation. Original pull request: #194.
mp911de
force-pushed
the
issue/DATAREDIS-503
branch
from
8ef926f to
3efc8ec
Compare
|
Rolled back the generic type change. Thanks for checking. |
mp911de
pushed a commit
that referenced
this pull request
May 2, 2016
…nverter. We now support mapping of simple and complex types to Redis HASH structures applying the same structure as the Repository support. This allows Object to hash Mapping and its direct usage via RedisTemplate without the need of explicitly having to use the repository abstraction. Original pull request: #194.
mp911de
added a commit
that referenced
this pull request
May 2, 2016
Rename ConvertingHashMapper to ObjectHashMapper. Add fromHash with type to avoid casting. Add JavaDoc documentation to HashMapper. Add hash mapping to reference documentation. Original pull request: #194.
|
That's merged with c1476b7. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
We now support mapping of simple and complex types to Redis HASH structures applying the same structure as the Repository support. This allows
Objectto hash mapping and its direct usage viaRedisTemplatewithout the need of explicitly having to use the repository abstraction.