gaodayue commented on a change in pull request #6402: Thread-safe QueryMetrics URL: https://github.com/apache/incubator-druid/pull/6402#discussion_r227255825
########## File path: processing/src/main/java/org/apache/druid/query/QueryMetrics.java ########## @@ -92,7 +94,7 @@ * 100% guarantee of compatibility, because methods could not only be added to QueryMetrics, existing methods could also * be changed or removed. * - * <p>QueryMetrics is designed for use from a single thread, implementations shouldn't care about thread-safety. + * <p><b>All implementations of QueryMetrics should be Thread-safe.</b> * * * Adding new methods to QueryMetrics Review comment: If needed, we can do the renaming in a separate PR. I want to keep this PR small and easy to review. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on 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
