[GitHub] [apisix] idbeta commented on pull request #2273: test: add integration test

2020-10-09 Thread GitBox
idbeta commented on pull request #2273: URL: https://github.com/apache/apisix/pull/2273#issuecomment-705995882 There are too many questions about this pr, close it, let me create another pr,thank u all. This is an automated

[GitHub] [apisix] idbeta commented on pull request #2273: test: add integration test

2020-10-09 Thread GitBox
idbeta commented on pull request #2273: URL: https://github.com/apache/apisix/pull/2273#issuecomment-705995882 There are too many questions about this pr, close it, let me create another pr,thank u all. This is an automated

[GitHub] [apisix] idbeta commented on pull request #2273: test: add integration test

2020-09-30 Thread GitBox
idbeta commented on pull request #2273: URL: https://github.com/apache/apisix/pull/2273#issuecomment-701258413 When users use API​​SIX, some problems such as setting too many routes to cause high memory usage after *long time* running, or problems caused by abnormal operation of related

[GitHub] [apisix] idbeta commented on pull request #2273: test: add integration test

2020-09-27 Thread GitBox
idbeta commented on pull request #2273: URL: https://github.com/apache/apisix/pull/2273#issuecomment-699746139 > The `t/integration` is confusing. Actually almost `*.t` files are for integration test, and only a few of them are unit tests. > Is there any different between the Python

[GitHub] [apisix] idbeta commented on pull request #2273: test: add integration test

2020-09-25 Thread GitBox
idbeta commented on pull request #2273: URL: https://github.com/apache/apisix/pull/2273#issuecomment-698199747 > `t/pycases` is not a good name, how about `t/integration` ? got it and fixed This is an automated

[GitHub] [apisix] idbeta commented on pull request #2273: test: add integration test

2020-09-24 Thread GitBox
idbeta commented on pull request #2273: URL: https://github.com/apache/apisix/pull/2273#issuecomment-698199747 > `t/pycases` is not a good name, how about `t/integration` ? got it and fixed This is an automated