search

steve0511 / resty-redis-cluster

Openresty lua client for redis cluster.
Apache License 2.0
376 stars 134 forks source link

Cache slots info & Removing "master_nodes" module variable #96

Open shuxiao9058 opened 2 years ago

shuxiao9058 commented 2 years ago

refer #86

also fix a potential bug caused with limit pool when tcp connect failed with too many waiting connect operations.

more details:

https://github.com/openresty/lua-nginx-module#tcpsockconnect

shuxiao9058 commented 2 years ago

@steve0511 please have a review, thanks.

shuxiao9058 commented 2 years ago

@toredash any suggestion about this pr?

toredash commented 2 years ago

I quickly looked over it, seems correct but I only spent a solid minute to review it.

I haven't used this tool for a few years now.

Did you so any noticeable performance benefit after this change?