gianm commented on a change in pull request #8114: Fix race between canHandle() and addSegment() in StorageLocation URL: https://github.com/apache/incubator-druid/pull/8114#discussion_r306579287
########## File path: indexing-service/src/main/java/org/apache/druid/indexing/worker/IntermediaryDataManager.java ########## @@ -63,10 +64,10 @@ * This class manages intermediary segments for data shuffle between native parallel index tasks. * In native parallel indexing, phase 1 tasks store segment files in local storage of middleManagers * and phase 2 tasks read those files via HTTP. - * + * <p> Review comment: I think the reason we usually avoid them is that they look silly, and also aren't adding anything useful since Druid javadocs are mostly read in source form anyway. It's not a big deal either way. Maybe one day someone will add a style rule for it and get rid of them all. ---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
