lnbest0707-uber commented on PR #16494:
URL: https://github.com/apache/pinot/pull/16494#issuecomment-3221006399

   > E.g., 11010005 means the 5th partition of the 11th topic (in the topic 
map) and it is a backfill topic (so 01). This is the way we can detect a 
backfill topic.
   
   With this naming format, if we remove a topic, it might affect other topic's 
pinot partition naming format.
   Let's say there's we have 11th topic A as backfill topic, 1101000x, 12th 
topic B also backfill topic, 1201000x. Then when you remove topicA, how should 
we naming the topic B. With the naming format, it would become 1101000x, which 
would be messed up.
   
   >   Today in multi-topic, we use the index of the topic in the streamConfig 
map to map a topic to a int. The above proposal uses the same idea. Any issue 
with continuing today's practice.
   
   As above example, the issue is about **removing topics**. This index mapping 
makes topic partition ID rely on its **position**. And when other topics got 
removed, the position would change.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to