leventov commented on issue #9134: Fix concurrency of ComplexMetrics.java URL: https://github.com/apache/druid/pull/9134#issuecomment-574679563 @capistrant ideally there should be concurrent unit tests, but I wouldn't obsess about it here. (I should probably change the language of the corresponding checklist item because creating correct concurrent unit tests is so hard that it should be justified only in a minority of cases.)
---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org