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

Roman Shtykh commented on IGNITE-3066:
--------------------------------------

Alexey, thank you.

1) Done
2) Yes, we can remove {{else}}, however type error is related to {{if}} clause, 
so I would leave it as it is now.
3) It can be public or package-private only. Let's leave it public.
4) Done
5) Good idea. I will put a link to the Redis command reference. I think 
referring is better because it has a detailed description of commands, and the 
documentation there may improve with even more details.
6) Let's do it. Done.
7) {{GridTcpRestParser}} handles {{ByteBuffer}}. What are the concerns?
8) Thank you for pointing at it! I used it in the initial implementation before 
wrapping over REST. Removed.
9) Renamed.


> Set of Redis commands that can be easily implemented via existing REST 
> commands
> -------------------------------------------------------------------------------
>
>                 Key: IGNITE-3066
>                 URL: https://issues.apache.org/jira/browse/IGNITE-3066
>             Project: Ignite
>          Issue Type: Sub-task
>            Reporter: Roman Shtykh
>            Assignee: Roman Shtykh
>
> As the 1st iteration of IGNITE-2788 the following commands can be implemented 
> via existing REST commands,
> GET
> MGET
> SET
> MSET
> INCR
> DECR
> INCRBY
> DECRBY
> APPEND
> STRLEN
> GETSET
> SETRANGE
> GETRANGE
> DEL
> EXISTS
> DBSIZE
> http://redis.io/commands



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to