wwbmmm commented on PR #3498: URL: https://github.com/apache/brpc/pull/3498#issuecomment-5643470856
Sorry for the copilot spam above. This was caused by my automatic agent. I think this feature can be more general, for example "limit server connections" , not specific to the redis protocol. As your document says: > Set `ServerOptions.redis_max_connections` to limit simultaneous connections on a Redis-only public listener. The default value is 0, which disables the limit. A non-zero value requires `redis_service` to be set, `enabled_protocols` to be exactly `"redis"`, builtin services to be disabled, and no RPC or other protocol services to share the Server. Since you limit the `enabled_protocol` to "redis", the redis connection count is equal to the whole server connection count, right? So we just need to limit the server connection count. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
