-1 binding. my reason: performance is not the first goal of Apache APISIX. We should give priority to satisfying user functions.
And do not submit a PR and merge before reaching a consensus on the mailing list. This is a violation of the apache way. Thanks, Ming Wen, Apache APISIX & Apache SkyWalking Twitter: _WenMing YuanSheng Wang <membp...@apache.org> 于2020年8月4日周二 下午7:08写道： > On Sat, Aug 1, 2020 at 8:01 PM agile6v <agil...@apache.org> wrote: > >> Hi YuanSheng >> >> AFAIK, If these plugins are not configured through the admin api, they >> will not be enabled by default. Also How is the 10% performance loss >> verified? >> > > that is the benchmark result: > > enable all plugin: the QPS is about 20118 req/sec. > remove the nginx.conf of plugin `proxy-cache` and `proxy-mirror`: the QPS > is about 22448 req/sec. > > [image: image.png] > > >> >> Thanks. >> >> On 2020/08/01 05:53:01, YuanSheng Wang <membp...@apache.org> wrote: >> > Hi: >> > >> > if we disable plugin `proxy-cache` and `proxy-mirror` and remove them >> from >> > `conf/nginx.conf`, >> > It will increase APISIX performance by 10%. >> > >> > proxy-cache conf [1] >> > proxy-mirror conf [2] >> > >> > I think this is useful for most users. >> > >> > [1] https://github.com/apache/apisix/blob/master/bin/apisix#L462 >> > [2] https://github.com/apache/apisix/blob/master/bin/apisix#L490 >> > >> > -- >> > >> > *MembPhis* >> > My GitHub: https://github.com/membphis >> > Apache APISIX: https://github.com/apache/incubator-apisix >> > >> > > > -- > > *MembPhis* > My GitHub: https://github.com/membphis > Apache APISIX: https://github.com/apache/incubator-apisix >
