JingsongLi commented on PR #9831: URL: https://github.com/apache/paimon/pull/9831#issuecomment-5772177951
My concrete recommendation is to keep the current storage layout and replace the zero-filled sparse temporary file with a seekable file-like reader backed by range reads. Use the persisted index to prefetch likely ranges; when the decoder requests uncached bytes, fetch the actual bytes from the video payload. This makes decoding correctness independent of the fixed extra-GOP and two-keyframe retry heuristics. Also apply the two inline fixes: [exclude `free`/`skip` padding bodies from initialization ranges](https://github.com/apache/paimon/pull/9831#discussion_r4068931136) and [seek with `backward=True`](https://github.com/apache/paimon/pull/9831#discussion_r4068931132). I do not see a demonstrated need to add DTS or GOP-length fields, or to redesign the on-disk layout. -- 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]
