npawar commented on pull request #6466:
URL: https://github.com/apache/incubator-pinot/pull/6466#issuecomment-763855321


   > It would help to also describe the following in the PR:
   > 
   > * What is the use case that requires empty segments?
   > * For a table with time based realtime segment creation, we might end up 
creating too many empty segments (if event freq << flush frequency)? If so, we 
should add checks in the code to not do so.
   
   Both these are in the description and discussions of the issue linked in the 
description. Check it out #6453
    


----------------------------------------------------------------
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.

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