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

Reply via email to