[GitHub] [pinot] Jackie-Jiang commented on pull request #8402: add segment size metric on segment addition rather than push

[GitBox <gi...@apache.org>]

Jackie-Jiang commented on pull request #8402:
URL: https://github.com/apache/pinot/pull/8402#issuecomment-1077891649


   This won't work properly because when the server restarts, there is no guarantee which segment is loaded first.
   If it is okay to track the compressed segment size (segment size in the deep store), @timsants is working on adding the compressed segment size to the ZK metadata in #8358. Then you may use that segment size which is always the actual compressed file size


-- 
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-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org