jianzhenwu opened a new pull request, #13121:
URL: https://github.com/apache/gluten/pull/13121

   ## What changes are proposed in this pull request?
   
   - Prevent compact row IDs from encoding unrepresentable page offsets by 
moving subsequent rows to a new page when needed.
   - Validate page numbers and row boundaries before reading rows during 
eviction.
   - Add a regression test for an oversized row followed by a small row. It 
verifies the page rollover and that `stop()` successfully writes both rows.
   
   ## How was this patch tested?
   
   - Built `velox_sort_shuffle_writer_test`.
   - Ran `./build/velox/tests/velox_sort_shuffle_writer_test` — 2 tests passed.
   - Confirmed the new regression test failed before the fix because the writer 
did not roll over to a second page.
   
   ## Was this patch authored or co-authored using generative AI tooling?
   
   Yes. `Generated-by: OpenAI Codex (GPT-6)`


-- 
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