Jackie-Jiang commented on a change in pull request #7325:
URL: https://github.com/apache/pinot/pull/7325#discussion_r692545485
##########
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:
This seems still over 120 characters. Shall we try to make all of them
under 120 characters if possible? Same for other files
--
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
