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/06/21 01:21:22 UTC

[GitHub] [incubator-druid] AlexanderSaydakov opened a new pull request #7938: locking in Theta sketch buffer aggregator

AlexanderSaydakov opened a new pull request #7938: locking in Theta sketch buffer aggregator
URL: https://github.com/apache/incubator-druid/pull/7938
 
 
   Added locking to theta buffer aggregator, factored out common locking code.
   
   It seems to me that this locking was missing from the very beginning, but was not quite necessary so far.
   As I understand it might become important in the future or is already needed in some particular use cases. This was discussed in several pull requests.
   For instance:
   https://github.com/apache/incubator-druid/pull/6581
   https://github.com/apache/incubator-druid/pull/5002
   https://github.com/apache/incubator-druid/pull/5148
   

----------------------------------------------------------------
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