You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2019/05/18 13:08:35 UTC

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

sanastas commented on issue #7676: Add OakIncrementalIndex to Druid
URL: https://github.com/apache/incubator-druid/pull/7676#issuecomment-493675946
 
 
   Hi @b-slim ,
   
   First, it is great you are taking a look :) , thanks!
   
   Second, we are trying now different Memory Allocators and Memory Managers. The implementation is not final, so we allow ourselves to make it easier and just to check the performance. The locks you see are for singleton initialization just upon a start. Most-likely, Memory Allocator/Pool will not continue to be singleton.
   Anyway, there are currently no locks on the hot path, and no locks are going to be there. 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org