bbr限流实现 maxInFlight 方法可能有误

[go-kratos]

bbr限流实现 maxInFlight 方法可能有误

[haiyux]

aegis/ratelimit/bbr/bbr.go

Line 235 in 56c0fb5

return int64(math.Floor(float64(l.maxPASS()*l.minRT()*l.bucketPerSecond)/1000.0) + 0.5)

最后加的那个0.5是不是放错位置了。Floor函数返回的已经是一个整数值，在加 0.5 后，会被最外层的 int64() 截断吧

+0.5 是为了向上取整

[DZGodly]

aegis/ratelimit/bbr/bbr.go

Line 235 in 56c0fb5

return int64(math.Floor(float64(l.maxPASS()*l.minRT()*l.bucketPerSecond)/1000.0) + 0.5)

最后加的那个0.5是不是放错位置了。Floor函数返回的已经是一个整数值，在加 0.5 后，会被最外层的 int64() 截断吧

+0.5 是为了向上取整

加错位置了吧，应该先加 0.5 再进行 Floor 操作吧

[juwell]

why not use math.Ceil() to get value greater than or equal to x ?

the comment for Ceil

// Ceil returns the least integer value greater than or equal to x.
//
// Special cases are:
//	Ceil(±0) = ±0
//	Ceil(±Inf) = ±Inf
//	Ceil(NaN) = NaN
func Ceil(x float64) float64

[lxkaka]

not same with +0.5