membphis commented on PR #13670:
URL: https://github.com/apache/apisix/pull/13670#issuecomment-4932328000

   The attributes table in `docs/en/latest/plugins/ai-rate-limiting.md` is 
missing `redis_keepalive_pool`, and it documents `redis_keepalive_timeout` as 
sentinel-only (default 60000).
   
   However, the `redis` and `redis-cluster` schema branches also define 
`redis_keepalive_timeout` (default 10000) and `redis_keepalive_pool` (default 
100), and both fields are forwarded to limit-count by the schema-driven copy in 
`transform_limit_conf`, so they take effect for all three redis policies.
   
   Please complete the doc table to cover these two fields for `redis` / 
`redis-cluster` as well, with the correct defaults.


-- 
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]

Reply via email to