(#2120) GeoRadius support store and storedist option

[yangbodong22011]

Resolves #2120
Because STORE and STOREDIST return Integer, they are not compatible with GEORADIUS return MultiBulk, so new commands georadiusStore and georadiusByMemberStore have been added.

[startjava]

georadiusStore and georadiusByMemberStore please which version add?? thank you !

…

------------------&nbsp;原始邮件&nbsp;------------------ 发件人:&nbsp;"杨博东"<notifications@github.com&gt;; 发送时间:&nbsp;2020年2月14日(星期五) 下午3:53 收件人:&nbsp;"xetorthio/jedis"<jedis@noreply.github.com&gt;; 抄送:&nbsp;"Subscribed"<subscribed@noreply.github.com&gt;; 主题:&nbsp;[xetorthio/jedis] (#2120) GeoRadius support store and storedist option (#2140) Resolves #2120 Because STORE and STOREDIST return Integer, they are not compatible with GEORADIUS return MultiBulk, so new commands georadiusStore and georadiusByMemberStore have been added. You can view, comment on, or merge this pull request online at: &nbsp; #2140 Commit Summary (#2120) GeoRadius support store and storedist option File Changes M src/main/java/redis/clients/jedis/BinaryClient.java (11) M src/main/java/redis/clients/jedis/BinaryJedis.java (16) M src/main/java/redis/clients/jedis/BinaryJedisCluster.java (22) M src/main/java/redis/clients/jedis/BinaryShardedJedis.java (14) M src/main/java/redis/clients/jedis/Client.java (10) M src/main/java/redis/clients/jedis/Jedis.java (16) M src/main/java/redis/clients/jedis/JedisCluster.java (22) M src/main/java/redis/clients/jedis/PipelineBase.java (28) M src/main/java/redis/clients/jedis/ShardedJedis.java (14) M src/main/java/redis/clients/jedis/commands/BinaryJedisClusterCommands.java (6) M src/main/java/redis/clients/jedis/commands/BinaryJedisCommands.java (6) M src/main/java/redis/clients/jedis/commands/BinaryRedisPipeline.java (6) M src/main/java/redis/clients/jedis/commands/JedisClusterCommands.java (6) M src/main/java/redis/clients/jedis/commands/JedisCommands.java (6) M src/main/java/redis/clients/jedis/commands/RedisPipeline.java (6) M src/main/java/redis/clients/jedis/params/GeoRadiusParam.java (27) M src/test/java/redis/clients/jedis/tests/commands/GeoCommandsTest.java (67) Patch Links: https://github.com/xetorthio/jedis/pull/2140.patch https://github.com/xetorthio/jedis/pull/2140.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or unsubscribe.

[yangbodong22011]

@startjava You need to wait for PR to be reviewed and then released.

[sazzad16]

Shouldn't GEORADIUS with STORE/STOREDIST be considered as multi key operations?

[yangbodong22011]

@sazzad16 Yes, GEORADIUS with STORE/STOREDIST should be considered as multi key operations.

I plan to modify my PR to put georadiusStore and georadiusByMemberStore in MultiKeyCommands.java and MultiKeyJedisClusterCommands.java and MultiKeyPipelineBase .java and their binary versions.

Before that, I want to confirm whether adding georadiusStore and georadiusByMemberStoreis a good way？

I hope you have some suggestions.

[sazzad16]

@yangbodong22011

Before that, I want to confirm whether adding georadiusStore and georadiusByMemberStoreis a good way？

I'm mostly okay with current approach.

[yangbodong22011]

I implementedgeoradiusStoreandgeoradiusByMemberStoreas multi key commands.