fix double precision ，module bloomfilter double to string

heyone-top commented 5 months ago

fix double precision ，module double to string 当我们使用redisshake迁移的时候发现 bloomfilter 模块无法迁移精度超过 0.000001的 ratio字段，因为最后他会白这个精度 ratio修改成0，这会导致记载 boomfilter过滤器的实例无法 bgsave，因为 ratio默认不会为0，最后经过排查发现是 redisshake精度转换有问题。 tips： ratio每次库容都会降低一个数量级。迁移后目标端会成为这种情况。

127.0.0.1:6379> bf.debug myboom
1) "size:1372567"
2) "bytes:4194304 bits:33554432 hashes:24 hashwidth:64 capacity:1000200 size:1000200 ratio:0"
3) "bytes:16777216 bits:134217728 hashes:26 hashwidth:64 capacity:3683950 size:372367 ratio:0"

当然我也查看了一下调用此函数的其他地方，tairzset 的rewrite 功能也会导致精度丧失。我觉得这是一个比较严重的 bug。

CLAassistant commented 5 months ago

All committers have signed the CLA.

suxb201 commented 5 months ago

%g 并不会提高输出精度，且出现科学计数法时会导致出错。考虑下：%.15f

heyone-top commented 5 months ago

%g 并不会提高输出精度，且出现科学计数法时会导致出错。考虑下：%.15f

嗯，我是看了对应的调用地方都处理了科学计数法，不过这个是float64类型，你这个表示也是没问题的。double小数位：2^52 = 4503599627370496，一共16位，所以 %.15f 比较好。我再更新一下

tair-opensource / RedisShake

fix double precision ，module bloomfilter double to string #830