[
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)