[GitHub] [pinot] Jackie-Jiang commented on pull request #10433: Add support to create realtime segment in local

["Jackie-Jiang (via GitHub)" <gi...@apache.org>]

Jackie-Jiang commented on PR #10433:
URL: https://github.com/apache/pinot/pull/10433#issuecomment-1472457463

   This is not the correct way to create segment to be pushed to realtime table. We intentionally not use the LLC segment name format to prevent it from messing up the actual consuming segment management (start/end offset won't exist in the offline created segment). To create partitioned segments, the partition info should be set within the table config, instead of hacking the segment name.


-- 
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-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org