SynchronizedUnion wrapper in Theta module was removed since it was not compatible with the new API of Union. Synchronization was added to aggregate() and get() methods in SketchAggregator instead. Also I noticed that there is no synchronization in the SketchBufferAggregator like in other modules. Is this an oversight or maybe the synchronization is not needed in other modules as well? @will-lauer do you happen to know?
[ Full content available at: https://github.com/apache/incubator-druid/pull/6381 ] This message was relayed via gitbox.apache.org for [email protected]
