[GitHub] [apisix] membphis commented on a change in pull request #2340: feature: limit-count use redis cluster

2020-10-13 Thread GitBox
membphis commented on a change in pull request #2340: URL: https://github.com/apache/apisix/pull/2340#discussion_r503707079 ## File path: apisix/plugins/limit-count/limit-count-redis-cluster.lua ## @@ -0,0 +1,130 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under

[GitHub] [apisix] membphis commented on a change in pull request #2340: feature: limit-count use redis cluster

2020-10-12 Thread GitBox
membphis commented on a change in pull request #2340: URL: https://github.com/apache/apisix/pull/2340#discussion_r503707079 ## File path: apisix/plugins/limit-count/limit-count-redis-cluster.lua ## @@ -0,0 +1,130 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under

[GitHub] [apisix] membphis commented on a change in pull request #2340: feature: limit-count use redis cluster

2020-10-12 Thread GitBox
membphis commented on a change in pull request #2340: URL: https://github.com/apache/apisix/pull/2340#discussion_r503374436 ## File path: t/plugin/limit-count-redis-cluster.t ## @@ -0,0 +1,228 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contri

[GitHub] [apisix] membphis commented on a change in pull request #2340: feature: limit-count use redis cluster

2020-10-12 Thread GitBox
membphis commented on a change in pull request #2340: URL: https://github.com/apache/apisix/pull/2340#discussion_r503374195 ## File path: t/plugin/limit-count-redis-cluster.t ## @@ -0,0 +1,228 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contri

[GitHub] [apisix] membphis commented on a change in pull request #2340: feature: limit-count use redis cluster

2020-10-12 Thread GitBox
membphis commented on a change in pull request #2340: URL: https://github.com/apache/apisix/pull/2340#discussion_r503373211 ## File path: bin/apisix ## @@ -190,6 +190,7 @@ http { lua_shared_dict balancer_ewma10m; lua_shared_dict balancer_ewma_locks 10m;

[GitHub] [apisix] membphis commented on a change in pull request #2340: feature: limit-count use redis cluster

2020-10-10 Thread GitBox
membphis commented on a change in pull request #2340: URL: https://github.com/apache/apisix/pull/2340#discussion_r502795882 ## File path: rockspec/apisix-master-0.rockspec ## @@ -54,6 +54,7 @@ dependencies = { "skywalking-nginx-lua-plugin = 1.0-0", "base64 = 1.5-2",

[GitHub] [apisix] membphis commented on a change in pull request #2340: feature: limit-count use redis cluster

2020-10-10 Thread GitBox
membphis commented on a change in pull request #2340: URL: https://github.com/apache/apisix/pull/2340#discussion_r502794290 ## File path: rockspec/apisix-master-0.rockspec ## @@ -54,6 +54,7 @@ dependencies = { "skywalking-nginx-lua-plugin = 1.0-0", "base64 = 1.5-2",

[GitHub] [apisix] membphis commented on a change in pull request #2340: feature: limit-count use redis cluster

2020-10-10 Thread GitBox
membphis commented on a change in pull request #2340: URL: https://github.com/apache/apisix/pull/2340#discussion_r502787623 ## File path: apisix/plugins/limit-count/limit-count-redis-cluster.lua ## @@ -0,0 +1,127 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under

[GitHub] [apisix] membphis commented on a change in pull request #2340: feature: limit-count use redis cluster

2020-10-10 Thread GitBox
membphis commented on a change in pull request #2340: URL: https://github.com/apache/apisix/pull/2340#discussion_r502777237 ## File path: apisix/plugins/limit-count.lua ## @@ -119,6 +148,11 @@ local function create_limit_obj(conf) conf.count, con

[GitHub] [apisix] membphis commented on a change in pull request #2340: feature: limit-count use redis cluster

2020-10-09 Thread GitBox
membphis commented on a change in pull request #2340: URL: https://github.com/apache/apisix/pull/2340#discussion_r502743834 ## File path: doc/zh-cn/plugins/limit-count.md ## @@ -26,17 +26,18 @@ ## 参数 -| 名称 | 类型| 必选项 | 默认值 | 有效值

[GitHub] [apisix] membphis commented on a change in pull request #2340: feature: limit-count use redis cluster

2020-10-09 Thread GitBox
membphis commented on a change in pull request #2340: URL: https://github.com/apache/apisix/pull/2340#discussion_r502721349 ## File path: apisix/plugins/limit-count/limit-count-redis-cluster.lua ## @@ -0,0 +1,139 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under

[GitHub] [apisix] membphis commented on a change in pull request #2340: feature: limit-count use redis cluster

2020-10-09 Thread GitBox
membphis commented on a change in pull request #2340: URL: https://github.com/apache/apisix/pull/2340#discussion_r501548890 ## File path: apisix/plugins/limit-count.lua ## @@ -132,7 +132,9 @@ function _M.check_schema(conf) end if conf.policy == "redis-cluster" then

[GitHub] [apisix] membphis commented on a change in pull request #2340: feature: limit-count use redis cluster

2020-10-09 Thread GitBox
membphis commented on a change in pull request #2340: URL: https://github.com/apache/apisix/pull/2340#discussion_r502279935 ## File path: apisix/plugins/limit-count/limit-count-redis-cluster.lua ## @@ -0,0 +1,139 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under

[GitHub] [apisix] membphis commented on a change in pull request #2340: feature: limit-count use redis cluster

2020-10-08 Thread GitBox
membphis commented on a change in pull request #2340: URL: https://github.com/apache/apisix/pull/2340#discussion_r501552477 ## File path: apisix/plugins/limit-count/limit-count-redis-cluster.lua ## @@ -0,0 +1,138 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under

[GitHub] [apisix] membphis commented on a change in pull request #2340: feature: limit-count use redis cluster

2020-10-08 Thread GitBox
membphis commented on a change in pull request #2340: URL: https://github.com/apache/apisix/pull/2340#discussion_r501548890 ## File path: apisix/plugins/limit-count.lua ## @@ -132,7 +132,9 @@ function _M.check_schema(conf) end if conf.policy == "redis-cluster" then

[GitHub] [apisix] membphis commented on a change in pull request #2340: feature: limit-count use redis cluster

2020-10-08 Thread GitBox
membphis commented on a change in pull request #2340: URL: https://github.com/apache/apisix/pull/2340#discussion_r501548890 ## File path: apisix/plugins/limit-count.lua ## @@ -132,7 +132,9 @@ function _M.check_schema(conf) end if conf.policy == "redis-cluster" then

[GitHub] [apisix] membphis commented on a change in pull request #2340: feature: limit-count use redis cluster

2020-10-05 Thread GitBox
membphis commented on a change in pull request #2340: URL: https://github.com/apache/apisix/pull/2340#discussion_r499664897 ## File path: rockspec/apisix-master-0.rockspec ## @@ -54,6 +54,7 @@ dependencies = { "skywalking-nginx-lua-plugin = 1.0-0", "base64 = 1.5-2",