swaminathanmanish commented on PR #11650: URL: https://github.com/apache/pinot/pull/11650#issuecomment-1742225871
> Yes, we still want to use the global sequence ID which isn't supported by the `FixedSegmentNameGenerator`. Another option here is to update `FixedSegmentNameGenerator` to support appending the sequence ID. Let me know if that approach is preferred. Thanks ! Lets go with SimpleSegmentNameGenerator itself. FixedSegmentNameGenerator can just be for a specific names set by the user, without the other attributes. -- 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
