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

Reply via email to