Jackie-Jiang commented on issue #10237: URL: https://github.com/apache/pinot/issues/10237#issuecomment-1431848472
> * We are proposing to prevent deletion of segments in CONSUMING state. ERROR state is not included in this I don't follow this. Are you saying you want to check EV to decide whether a segment can be deleted? That doesn't sound correct to me. > * Can we change PAUSE to include error segments (optionally)? All it would do is to simply remove the segment, and change the IDEALSTATE to make sure that it is consistent with missing CONSUMING segments. wdyt? This proposal is essentially making PAUSE to delete the consuming segment. IMO this behavior is not intuitive to the user. I still suggest allowing the primitive operation of deleting a segment. To prevent mis-operation, adding a flag is the preferred solution. -- 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-unsubscr...@pinot.apache.org 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
