[GitHub] [druid] clintropolis commented on issue #9736: HLLSketchMerge aggregator failing for some metrics after upgrade to v0.18

[GitBox <gi...@apache.org>]

clintropolis commented on issue #9736:
URL: https://github.com/apache/druid/issues/9736#issuecomment-618054731


   Thanks for the segment and example query, I can reproduce this issue in the debugger with group by query and can confirm that it doesn't seem to affect timeseries queries, which means it is likely an issue with `HllSketchMergeBufferAggregator` or how it is being used, (timeseries is using `HllSketchMergeAggregator`).
   
   >The "concurrent" exception that was thrown makes me nervous :)
   
   This is an unfortunate presentation issue, that's not the actual exception on the historical, but the side effect of the broker parallel merging catching an error on an individual query. I'll see if I can try to improve this in the future to make it less confusing.


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



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