Re: [Proposal] Backport GEODE-9009 - Support for Redis DECRBY command

[Owen Nichols]

Sounds like this is worth tracking as a blocker, thanks for explaining the 
consequence of not including this.

On 3/12/21, 12:08 PM, "Raymond Ingles" <ring...@vmware.com> wrote:

    Hello –

     Putting forward the proposal to backport GEODE-9009 (Redis DECRBY command 
support) to support/1.14 branch,

            What does GEODE-9009 do?

              *   It adds unit/integration/dunit tests for the Redis DECRBY 
command
              *   It moves the DECRBY command to the ‘Supported’ category

           These changes are low-risk as they are limited entirely to the 
Geode’s Redis-compatibility subsystem and do not impact any other Geode code.

            Why do we need to backport these changes?

              *   These changes will allow users of the Geode 
compatibility-with-Redis subsystem to use the DECRBY command
              *   If we don't backport these changes to 1.14.0 then the Redis 
compatibility subsystem will have an awkward gap in support, as the related 
commands (DECR, INCR, INCRBY, INCRBYFLOAT, HINCRBY, HINCRBYFLOAT) are already 
supported.

            Reference PR: 
https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F6119&amp;data=04%7C01%7Conichols%40vmware.com%7C1d14b58e076843e3e12b08d8e5929cbe%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C1%7C0%7C637511765359297530%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=%2BECjGuu3fhZQB7awVYAHjI%2FJB7yDHAzvvXQ%2FRh4tG4U%3D&amp;reserved=0

                    Thanks,
                    Ray Ingles