+1
________________________________
From: Raymond Ingles <[email protected]>
Sent: Friday, March 12, 2021 4:29 PM
To: [email protected] <[email protected]>
Subject: [Proposal] Backport GEODE-9029 - Initial support for Redis SLOWLOG
command
Hello –
Putting forward the proposal to backport GEODE-9029 (Redis SLOWLOG command
support) to support/1.14 branch,
What does GEODE-9029 do?
* It adds unit/integration/dunit tests for the Redis SLOWLOG command
* It moves the SLOWLOG 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.
The version of SLOWLOG is essentially a stub, because (a) Geode does not track
such data the way Redis does, but (b) if the command is not supported, some
monitoring tools (like certain versions of the DataDog agent) can experience
internal errors that prevent monitoring of other, fully-supported statistics.
Why do we need to backport these changes?
* These changes will allow automated tools that monitor Redis
health (e.g. DataDog) to also monitor the health of the Geode
compatibility-with-Redis subsystem.
* If we don't backport these changes to 1.14.0 then use of Redis
compatibility will be impaired or unacceptable for several users
Reference PR:
https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F6131&data=04%7C01%7Cdarrel%40vmware.com%7Cabc49d94111a4905f31308d8e5cbf118%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C1%7C0%7C637512011374535216%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=mVb69kveoXsK8Cz9rewmJX%2BtVHirekGc0C1nih3S6F0%3D&reserved=0
Thanks,
Ray Ingles