spacewander commented on pull request #3758:
URL: https://github.com/apache/apisix/pull/3758#issuecomment-792443816
After apply this change:
```
diff --git apisix/plugins/traffic-split.lua apisix/plugins/traffic-split.lua
index eb7a2028..3d5e7d8f 100644
--- apisix/plugins/traffic-split.lua
+++ apisix/plugins/traffic-split.lua
@@ -315,8 +315,7 @@ function _M.access(conf, ctx)
end
local route_upstream_id = ctx.matched_route.value.upstream_id
- local rr_up, err = lrucache(weighted_upstreams, nil, new_rr_obj,
weighted_upstreams,
- route_upstream_id)
+ local rr_up, err = new_rr_obj(weighted_upstreams, route_upstream_id)
if not rr_up then
core.log.error("lrucache roundrobin failed: ", err)
return 500
```
many test cases are failed. Look like `new_rr_obj` corrupts the route's
configuration. This may happen once the cache is evicted.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
