Jackie-Jiang commented on a change in pull request #7325: URL: https://github.com/apache/pinot/pull/7325#discussion_r692578133
########## File path: pinot-core/src/main/java/org/apache/pinot/core/data/manager/InstanceDataManager.java ########## @@ -134,7 +134,8 @@ void reloadAllSegments(String tableNameWithType) ZkHelixPropertyStore<ZNRecord> getPropertyStore(); /** - * Returns the segment uploader, which uploads a llc segment to the destination place and returns the url of uploaded segment file. Servers utilize segment uploader to upload llc segment to segment store. + * Returns the segment uploader, which uploads a llc segment to the destination place and returns the url of uploaded segment file. Review comment: Suggest keeping the 120 because everyone has been following the code format -- 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. To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org