Thibault Charbonnier
Thibault Charbonnier
Just a thought, the OpenResty fork of mpx/lua-cjson has some nice fixes and [additions](https://github.com/openresty/lua-cjson#additions), including the ability to chose the encoding type of empty Lua tables with more granularity (empty...
Hi, > 1. Can you please provide more details if it's supported than how to set max connections and other connection details. Setting a maximum amount of concurrent connections is...
I cannot help you without seeing some code. Most likely, you are doing something wrong. OpenResty programming is tricky. Are you correctly closing opened connections? Or keeping them alive? Are...
@MaximeFrancoeur Sure, connection queuing is available since OpenResty 1.15.8.1. See the new `pool_size` and `backlog` options of the [tcpsock:connect](https://github.com/openresty/lua-nginx-module#tcpsockconnect) method.
@MaximeFrancoeur Connection pools are already maintained as per the default behaviour documented in the ngx_http_lua module's README. If you want more control over the connection pools, PRs are welcome.
@MaximeFrancoeur Pools are created by calling ngx_lua's [tcpsock:setkeepalive](https://github.com/openresty/lua-nginx-module#tcpsocksetkeepalive). If you use the Cluster module, it calls `cassandra:setkeepalive()` (see [here](https://github.com/thibaultcha/lua-cassandra/blob/master/lib/resty/cassandra/cluster.lua#L808)), which itself calls `host:setkeepalive()` (see [here](https://github.com/thibaultcha/lua-cassandra/blob/master/lib/cassandra/init.lua#L294)). By default, pools are maintained...
A note on my previous comment in this thread: > Setting a maximum amount of concurrent connections is not supported by OpenResty as of today. It is now supported as...
The retry logic and the timeouts all come from the Datastax drivers that existed circa 2015 when this driver was implemented. I won't be updating the logic myself, but thanks...
Contributions to this driver aiming at supporting cluster topology events are welcome. So would be improving `refresh()` to support calls at runtime (see https://github.com/Yelp/casper/pull/47), which would be a nice first...
@nbisson No news on this front. Contributions still welcome.