[GitHub] [kvrocks] enjoy-binbin opened a new pull request, #1656: GEOSEARCH* from is not optional, FROMLONLAT or FROMMEMBER must require

["enjoy-binbin (via GitHub)" <gi...@apache.org>]

enjoy-binbin opened a new pull request, #1656:
URL: https://github.com/apache/kvrocks/pull/1656

   Fix that FROMLONLAT or FROMMEMBER is required, otherwise we would
   have the following inconsistency:
   ```
   127.0.0.1:6379> GEOSEARCH src BYBOX 88 88 m asc
   (error) ERR exactly one of FROMMEMBER or FROMLONLAT can be specified for GEOSEARCH
   127.0.0.1:6379> GEOSEARCHSTORE dst src BYBOX 88 88 m asc
   (error) ERR exactly one of FROMMEMBER or FROMLONLAT can be specified for GEOSEARCHSTORE
   
   127.0.0.1:6666> GEOSEARCH src BYBOX 88 88 m asc
   (empty array)
   127.0.0.1:6666> GEOSEARCHSTORE dst src BYBOX 88 88 m asc
   (integer) 0
   ```
   
   Minor fix to minor bug.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [kvrocks] enjoy-binbin merged pull request #1656: GEOSEARCH* from is not optional, FROMLONLAT or FROMMEMBER must require

["enjoy-binbin (via GitHub)" <gi...@apache.org>]

enjoy-binbin merged PR #1656:
URL: https://github.com/apache/kvrocks/pull/1656


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org