Jackie-Jiang commented on pull request #7306: URL: https://github.com/apache/pinot/pull/7306#issuecomment-899720610
Allowing custom segment name from HTTP header requires one of the following 2 conditions: 1. Update the physical stored metadata file (for deep store URL push, need to push back the segment) 2. Deprecate `SegmentMetadata.getName()` and only use it when segment name is not available from the HTTP header (similar to `SegmentMetadata.getTableName()`. On the server side, the segment name should never be read from the metadata, but from the ZK path -- 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
