upthewaterspout commented on pull request #6405: URL: https://github.com/apache/geode/pull/6405#issuecomment-832264021
@sabbey37 - The main reason I left MemoryOverheadNativeRedisAcceptanceTest in there is so we can compare our numbers with native redis and see how close we are getting. It's also a framework and place to add new measurements - for example to measure the native redis overhead of sorted sets to compare with radish. So I like having the code checked in somewhere. I guess I'm ambivalent about running it in CI every time. It takes about 4 seconds. -- 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. For queries about this service, please contact Infrastructure at: [email protected]
