Jackie-Jiang commented on PR #15817: URL: https://github.com/apache/pinot/pull/15817#issuecomment-2923713457
I applied a commit to completely unify the logic of OFFLINE and REALTIME segment: - First try to get partition id from segment name (this can be useful for `OFFLINE` table in the future if we want to reduce ZK access) - Read ZK metadata to find partition id: - When `partitionColumn` is provided, use it to look up the partition metadata - When it is not provided, if there is only one partition column, use it - For non-strict-replica-group assignment, fall back to default partition id -- 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