[GitHub] [incubator-druid] b-slim commented on issue #7676: Add OakIncrementalIndex to Druid

2019-06-20 Thread GitBox
b-slim commented on issue #7676: Add OakIncrementalIndex to Druid URL: https://github.com/apache/incubator-druid/pull/7676#issuecomment-504099736 @sanastas thanks for this update. Would be nice if you have time/resources to post more information about how this experiments are conducted

[GitHub] [incubator-druid] b-slim commented on issue #7676: Add OakIncrementalIndex to Druid

2019-05-20 Thread GitBox
b-slim commented on issue #7676: Add OakIncrementalIndex to Druid URL: https://github.com/apache/incubator-druid/pull/7676#issuecomment-494027530 @sanastas that is simple not true, some of those locks are part of `allocate`

[GitHub] [incubator-druid] b-slim commented on issue #7676: Add OakIncrementalIndex to Druid

2019-05-17 Thread GitBox
b-slim commented on issue #7676: Add OakIncrementalIndex to Druid URL: https://github.com/apache/incubator-druid/pull/7676#issuecomment-493536012 Just one point, not sure what lock-free means here but when i do grep on synchronized i see the following and some of those locks are used as