request_rate_limiter.lua should use milliseconds

[brucelwl]

spring-cloud-gateway/spring-cloud-gateway-server-webflux/src/main/resources/META-INF/scripts/request_rate_limiter.lua

Line 8 in 9689187

local now = tonumber(ARGV[3]) or redis.call('TIME')[1]

redis.call('TIME')[1] returns seconds, is it more reasonable to use milliseconds

[spencergibb]

Can you explain any issue you are having?

[brucelwl]

Ensure that token generation is evenly distributed.

If the current limit threshold I set is 10000 per second, then 10 tokens should be generated every 1ms instead of waiting until 1 second to generate 10000 tokens again

[spencergibb]

Again, is this causing you some kind of problem or error?

[brucelwl]

@spencergibb
At present, I am not using this script in a real production environment. I am using the flow limiting script in Redisson, which processes in milliseconds. Recently, I am researching switching to this script and found that it uses seconds. In high traffic situations, the token generated here is not uniform enough

[spencergibb]

The bucket4j rate limiter has more options, have you looked at that?