• antirez's avatar
    Added GEORADIUS(BYMEMBER)_RO variants for read-only operations. · f8547e53
    antirez authored
    Issue #4084 shows how for a design error, GEORADIUS is a write command
    because of the STORE option. Because of this it does not work
    on readonly slaves, gets redirected to masters in Redis Cluster even
    when the connection is in READONLY mode and so forth.
    
    To break backward compatibility at this stage, with Redis 4.0 to be in
    advanced RC state, is problematic for the user base. The API can be
    fixed into the unstable branch soon if we'll decide to do so in order to
    be more consistent, and reease Redis 5.0 with this incompatibility in
    the future. This is still unclear.
    
    However, the ability to scale GEO queries in slaves easily is too
    important so this commit adds two read-only variants to the GEORADIUS
    and GEORADIUSBYMEMBER command: GEORADIUS_RO and GEORADIUSBYMEMBER_RO.
    The commands are exactly as the original commands, but they do not
    accept the STORE and STOREDIST options.
    f8547e53
server.h 87.9 KB