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 2020/01/15 14:19:15 UTC
[GitHub] [druid] leventov commented on issue #9134: Fix concurrency of
ComplexMetrics.java
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:
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