Jackie-Jiang commented on PR #9624: URL: https://github.com/apache/pinot/pull/9624#issuecomment-1284695996
But why do we want to use a separate periodic task to trigger the reload? We should just trigger the reload in `SegmentTierAssigner` which modify the tier for the segment. Once the `SegmentTierAssigner` changes the target tier for a segment, the new loaded segment will already be moved, and we want the already loaded segment to have the same behavior to be consistent. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
