[jira] [Commented] (STORM-2304) [storm-redis] Provide binary version of TupleMapper and corresponding mappers

["Jungtaek Lim (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/STORM-2304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15875201#comment-15875201 ] 

Jungtaek Lim commented on STORM-2304:
-------------------------------------

This issue doesn't look like easy as it seems, because of backward compatibility issue.

In string version of methods, Jedis and JedisCluster share interface so easy to pick interface and go with common code, but binary version of methods are not.

I filed STORM-2369 to cover only State change, since it doesn't make backward compatibility issue.
(Maybe need to provide a tool to convert Base64 encoded State to origin...)

> [storm-redis] Provide binary version of TupleMapper and corresponding mappers
> -----------------------------------------------------------------------------
>
>                 Key: STORM-2304
>                 URL: https://issues.apache.org/jira/browse/STORM-2304
>             Project: Apache Storm
>          Issue Type: Improvement
>          Components: storm-redis
>            Reporter: Jungtaek Lim
>            Assignee: Jungtaek Lim
>
> For now storm-redis only provides string version of mappers which makes issues with binary key, field, and value. While converting data, data can be messed, or at least there should be time / space consuming.
> Redis originally takes binary data, and also Jedis has binary methods set. It should be better that storm-redis supports binary data, and after this state is not needed to be converted with base64.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)