[GitHub] commons-compress issue #21: COMPRESS-388: Fix concurrent reads performance

2017-04-23 Thread kvr000
Github user kvr000 commented on the issue: https://github.com/apache/commons-compress/pull/21 Additionally, I was thinking about exposing the entry raw stream starting offset and length via public API so in case of need one can either map it into memory, directly access the raw data

[GitHub] commons-compress issue #21: COMPRESS-388: Fix concurrent reads performance

2017-04-23 Thread kvr000
Github user kvr000 commented on the issue: https://github.com/apache/commons-compress/pull/21 Update, improving few things: - made the fields private again - simplified to single read(long pos, ByteBuffer buf) method - allocating the instance buffer only for single byte

[GitHub] commons-compress issue #21: COMPRESS-388: Fix concurrent reads performance

2017-04-23 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-compress/pull/21 [![Coverage Status](https://coveralls.io/builds/11205671/badge)](https://coveralls.io/builds/11205671) Coverage increased (+0.04%) to 84.277% when pulling

[GitHub] commons-compress issue #21: COMPRESS-388: Fix concurrent reads performance

2017-04-23 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-compress/pull/21 [![Coverage Status](https://coveralls.io/builds/11201739/badge)](https://coveralls.io/builds/11201739) Coverage increased (+0.07%) to 84.303% when pulling

[GitHub] commons-compress issue #21: COMPRESS-388: Fix concurrent reads performance

2017-04-23 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-compress/pull/21 [![Coverage Status](https://coveralls.io/builds/11199449/badge)](https://coveralls.io/builds/11199449) Coverage decreased (-0.02%) to 84.214% when pulling

[GitHub] commons-compress issue #21: COMPRESS-388: Fix concurrent reads performance

2017-04-23 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-compress/pull/21 [![Coverage Status](https://coveralls.io/builds/11199449/badge)](https://coveralls.io/builds/11199449) Coverage decreased (-0.02%) to 84.214% when pulling