xiangfu0 commented on a change in pull request #7325:
URL: https://github.com/apache/pinot/pull/7325#discussion_r692557904
##########
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:
We have a checkstyle hard failure limit of 160.
For now, I break the line at 140, tried with 120, and it makes many
expressions look a bit ugly with intentions.
--
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
