[GitHub] [pinot] snleee commented on pull request #8067: Wire EmptySegmentPruner to routing config

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

snleee commented on pull request #8067:
URL: https://github.com/apache/pinot/pull/8067#issuecomment-1021528527


   @Jackie-Jiang We may need the empty segment pruner in the future for some other use case so it is probably good to keep the empty segment pruner config wired. Anyway, I like your idea to auto-enable empty segment pruner if Kinesis consumer is used. In that way, we can keep the backward compatibility. That will save some effort to enable the empty segment pruner for all tables using Kinesis consumers.
   
   @mqliang How do you think?


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