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/07/09 07:58:19 UTC
[GitHub] [incubator-druid] gianm commented on issue #8031: remove
unnecessary synchronization overhead from complex Aggregators
gianm commented on issue #8031: remove unnecessary synchronization overhead from complex Aggregators
URL: https://github.com/apache/incubator-druid/issues/8031#issuecomment-509534521
It sounds to me like it would work. Btw, I wonder how meaningful the overhead is of synchronization in the case of most complex aggregators -- is it noticeable or is it dwarfed by the complex aggrecomputations? If it is meaningful, a change like this would be good for the expected majority of cases (nonconcurrent access on historicals).
----------------------------------------------------------------
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