balancer_by_lua 模块是否可以增加 upstream 参数的调整

xbsura

hi @agentzh:

使用了一下新开发的 balancer_by_lua 接口, 很快很好用, 不过暴露的接口只有设置与重试, 如果不同的后端需要设置不同的超时时间就没法做了, 问下是否有实现的可能, 类似:

set_upstream_connent_timeout() 这种接口

3ks

agentzh

Hello!

2016-01-12 4:05 GMT-08:00 肖贝贝:
> 使用了一下新开发的 balancer_by_lua 接口, 很快很好用, 不过暴露的接口只有设置与重试,
> 如果不同的后端需要设置不同的超时时间就没法做了, 问下是否有实现的可能, 类似:
>
> set_upstream_connent_timeout() 这种接口
>

这个在技术上肯定是可以实现的，需要给 nginx 核心打一个很小的补丁，因为 nginx upstream 目前使用的超时设置是直接从
nginx 配置中读取的，不好做每请求的隔离。回头我可以找 nginx 官方团队的哥们聊一聊这个问题。

Regards,
-agentzh

xbsura

好, 那我们自己先看看做个补丁用着, 期待正式版

3ks

在 2016年1月13日星期三 UTC+8上午3:26:32，agentzh写道：

Hello!

2016-01-12 4:05 GMT-08:00 肖贝贝:
> 使用了一下新开发的 balancer_by_lua 接口, 很快很好用, 不过暴露的接口只有设置与重试,
> 如果不同的后端需要设置不同的超时时间就没法做了, 问下是否有实现的可能, 类似:
>
> set_upstream_connent_timeout() 这种接口
>

这个在技术上肯定是可以实现的，需要给 nginx 核心打一个很小的补丁，因为 nginx upstream 目前使用的超时设置是直接从
nginx 配置中读取的，不好做每请求的隔离。回头我可以找 nginx 官方团队的哥们聊一聊这个问题。

Regards,
-agentzh

xbsura

hi @agentzh:

看了一下代码, 超时的设置大概是通过:

ngx_add_timer(c->write, u->conf->connect_timeout);

类似的 timer 完成的, 设置的超时时间都是 upstream 的结构体定义的, 貌似是全局共享的, 在想可不可以给 request_t 加三个超时参数, 设置的时候, 如果有 request_t 的就用 request_t 的设置, 如果没有, 用 upstream 的

lua 里增加对 request_t 的参数设置

不知道这样搞有没有别的风险

3ks

在 2016年1月13日星期三 UTC+8上午3:26:32，agentzh写道：

Hello!

2016-01-12 4:05 GMT-08:00 肖贝贝:
> 使用了一下新开发的 balancer_by_lua 接口, 很快很好用, 不过暴露的接口只有设置与重试,
> 如果不同的后端需要设置不同的超时时间就没法做了, 问下是否有实现的可能, 类似:
>
> set_upstream_connent_timeout() 这种接口
>

这个在技术上肯定是可以实现的，需要给 nginx 核心打一个很小的补丁，因为 nginx upstream 目前使用的超时设置是直接从
nginx 配置中读取的，不好做每请求的隔离。回头我可以找 nginx 官方团队的哥们聊一聊这个问题。

Regards,
-agentzh

agentzh

Hello!

2016-01-12 22:21 GMT-08:00 肖贝贝:
> 看了一下代码, 超时的设置大概是通过:
>
> ngx_add_timer(c->write, u->conf->connect_timeout);
>
> 类似的 timer 完成的, 设置的超时时间都是 upstream 的结构体定义的, 貌似是全局共享的, 在想可不可以给 request_t
> 加三个超时参数, 设置的时候, 如果有 request_t 的就用 request_t 的设置, 如果没有, 用 upstream 的
>
> lua 里增加对 request_t 的参数设置
>

思路是对的，但不应直接给 ngx_http_request_t 结构体来添加这样的字段，毕竟这只是 upstream 自己用的东西，应该放在
ngx_http_upstream_t 里面才合理。

Regards,
-agentzh

xbsura

ok, 3ks

在 2016年1月13日星期三 UTC+8下午2:29:33，agentzh写道：

Hello!

2016-01-12 22:21 GMT-08:00 肖贝贝:
> 看了一下代码, 超时的设置大概是通过:
>
> ngx_add_timer(c->write, u->conf->connect_timeout);
>
> 类似的 timer 完成的, 设置的超时时间都是 upstream 的结构体定义的, 貌似是全局共享的, 在想可不可以给 request_t
> 加三个超时参数, 设置的时候, 如果有 request_t 的就用 request_t 的设置, 如果没有, 用 upstream 的
>
> lua 里增加对 request_t 的参数设置
>

思路是对的，但不应直接给 ngx_http_request_t 结构体来添加这样的字段，毕竟这只是 upstream 自己用的东西，应该放在
ngx_http_upstream_t 里面才合理。

Regards,
-agentzh