jadami10 opened a new issue, #16435: URL: https://github.com/apache/pinot/issues/16435
We are working on orchestrating our rolling restarts such that we try to force commit all segments that have been consuming for > X minutes. What we find is we pre-compute the consuming segments we want to force commit, but as we do this in batches and poll, we eventually hit `Cannot commit segments that are not in CONSUMING state.` introduced in #12088. It doesn't seem like this needs to be an error. It would be better to either ignore this case or just log a warning. But if a user is requesting to force commit segment S, and it's already committed, then we might as well skip it. -- 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]
