LindaSummer commented on PR #3551: URL: https://github.com/apache/kvrocks/pull/3551#issuecomment-4967243070
> The CDF algorithm in this PR differs from Redis’s implementation. Was this intentional? > > ``` > TDIGEST.CREATE t COMPRESSION 10 > TDIGEST.ADD t 0 1 2 ... 99 > TDIGEST.CDF t 20 40 50 60 80 > ``` > > Query RedisBloom this PR > 20 0.205 0.09 > 40 0.405 0.33 > 50 0.505 0.63 > 60 0.605 0.63 > 80 0.805 0.88 Hi @jihuayu , Thanks very much for your checking! ❤️ Kvrocks result should align with Redis's, at least should not have such a clear difference. I will investigate the problem and find the root cause. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
