[GitHub] wu-sheng commented on issue #2247: Backend streaming thread model improvement

2019-02-17 Thread GitBox
wu-sheng commented on issue #2247: Backend streaming thread model improvement URL: https://github.com/apache/incubator-skywalking/pull/2247#issuecomment-464595848 From the latest test in mesh scenario, we could work in 75% cpu for supporting 25k mesh telemetry. Which is a big improvement.

[GitHub] wu-sheng commented on issue #2247: Backend streaming thread model improvement

2019-02-15 Thread GitBox
wu-sheng commented on issue #2247: Backend streaming thread model improvement URL: https://github.com/apache/incubator-skywalking/pull/2247#issuecomment-464076731 Related to #2168 too. This is an automated message from the Ap

[GitHub] wu-sheng commented on issue #2247: Backend streaming thread model improvement

2019-02-15 Thread GitBox
wu-sheng commented on issue #2247: Backend streaming thread model improvement URL: https://github.com/apache/incubator-skywalking/pull/2247#issuecomment-464058449 Now, the CPU at no payload status, is about 5%-8%, even started in IDEA. Also, I wonder @hanahmily 's test results in real K8s

[GitHub] wu-sheng commented on issue #2247: Backend streaming thread model improvement

2019-02-15 Thread GitBox
wu-sheng commented on issue #2247: Backend streaming thread model improvement URL: https://github.com/apache/incubator-skywalking/pull/2247#issuecomment-464057017 I set UI jetty thread default number to 4 (down from 10). Also I provide a new setting in core module ``` core: defa

[GitHub] wu-sheng commented on issue #2247: Backend streaming thread model improvement

2019-02-14 Thread GitBox
wu-sheng commented on issue #2247: Backend streaming thread model improvement URL: https://github.com/apache/incubator-skywalking/pull/2247#issuecomment-463919644 @peng-yongsheng I have another thought is, do you need to control the thread num of Jetty? I found 10 threads for Jetty, but we

[GitHub] wu-sheng commented on issue #2247: Backend streaming thread model improvement

2019-02-14 Thread GitBox
wu-sheng commented on issue #2247: Backend streaming thread model improvement URL: https://github.com/apache/incubator-skywalking/pull/2247#issuecomment-463895637 @hanahmily Could you set comparison between this pr codebase vs the old cases in your blog? @peng-yongsheng Please revie

[GitHub] wu-sheng commented on issue #2247: Backend streaming thread model improvement

2019-02-14 Thread GitBox
wu-sheng commented on issue #2247: Backend streaming thread model improvement URL: https://github.com/apache/incubator-skywalking/pull/2247#issuecomment-463895172 @hanahmily Also the thread number could be override by system env with thread pool name + `_THREAD` suffix. Such as, for

[GitHub] wu-sheng commented on issue #2247: Backend streaming thread model improvement

2019-02-14 Thread GitBox
wu-sheng commented on issue #2247: Backend streaming thread model improvement URL: https://github.com/apache/incubator-skywalking/pull/2247#issuecomment-463894944 I run `jstack` at my local machine, and show this. - One `DataCarrier.REGISTER_L1.BulkConsumePool`, ``DataCarrier.REGISTER_L