[GitHub] [activemq-artemis] franz1981 commented on issue #2633: ARTEMIS-2317 Avoid long TTSP caused by Page::read using mmap read

2019-04-29 Thread GitBox
franz1981 commented on issue #2633: ARTEMIS-2317 Avoid long TTSP caused by Page::read using mmap read URL: https://github.com/apache/activemq-artemis/pull/2633#issuecomment-487471785 @qihongxu that's super interesting :O TBH I was expecting the patched version to be on par with very

[GitHub] [activemq-artemis] franz1981 commented on issue #2633: ARTEMIS-2317 Avoid long TTSP caused by Page::read using mmap read

2019-04-28 Thread GitBox
franz1981 commented on issue #2633: ARTEMIS-2317 Avoid long TTSP caused by Page::read using mmap read URL: https://github.com/apache/activemq-artemis/pull/2633#issuecomment-487406011 @qihongxu just curious: which value are you configured for page-max-concurrent-io?

[GitHub] [activemq-artemis] franz1981 commented on issue #2633: ARTEMIS-2317 Avoid long TTSP caused by Page::read using mmap read

2019-04-28 Thread GitBox
franz1981 commented on issue #2633: ARTEMIS-2317 Avoid long TTSP caused by Page::read using mmap read URL: https://github.com/apache/activemq-artemis/pull/2633#issuecomment-487370942 @qihongxu I'm getting similar results to yours :) This pr together with

[GitHub] [activemq-artemis] franz1981 commented on issue #2633: ARTEMIS-2317 Avoid long TTSP caused by Page::read using mmap read

2019-04-25 Thread GitBox
franz1981 commented on issue #2633: ARTEMIS-2317 Avoid long TTSP caused by Page::read using mmap read URL: https://github.com/apache/activemq-artemis/pull/2633#issuecomment-486547865 The CI has shown good results: is basically on par of master :+1: Now my concern is just to

[GitHub] [activemq-artemis] franz1981 commented on issue #2633: ARTEMIS-2317 Avoid long TTSP caused by Page::read using mmap read

2019-04-23 Thread GitBox
franz1981 commented on issue #2633: ARTEMIS-2317 Avoid long TTSP caused by Page::read using mmap read URL: https://github.com/apache/activemq-artemis/pull/2633#issuecomment-485841941 @clebertsuconic Please don't merge it yet:I need reviews and some help from you and @wy96f and @qihongxu