ContentSquare / chproxy

Open-Source ClickHouse http proxy and load balancer
https://www.chproxy.org/
MIT License
1.24k stars 256 forks source link

feat: add pool config for redis #409

Closed rfyiamcool closed 3 months ago

rfyiamcool commented 4 months ago

Description

add poolsize config for redis. the default pool size of the go-redis is small, when many concurrent request to redis, the size of the connection pool affects performance. if the redis pool is full, then all conn in pool is busy, we need to wait a while.

I think we customize the size according to the requirements.

Pull request type

Please check the type of change your PR introduces:

Further comments

mga-chka commented 4 months ago

please, check the failing tests

mga-chka commented 4 months ago

fyi, there are still some issues with the tests

rfyiamcool commented 3 months ago

fyi, there are still some issues with the tests

done, have fixed wrong unit-test.

mga-chka commented 3 months ago

@rfyiamcool fyi I've released a new version of chproxy