LuciferYang opened a new pull request, #9765:
URL: https://github.com/apache/paimon/pull/9765

   ### Purpose
   
   close #9764
   
   `(int) MemorySize.parse(chunkSize).getBytes()` silently truncated a 
legitimate `chunk-size` such as `"2g"` to a negative int, crashing the index 
writer deep inside the first chunk. The chunk size also becomes an eagerly 
allocated per-chunk buffer, so values beyond int range can never work.
   
   This PR validates the option and rejects anything above `Integer.MAX_VALUE` 
with a message naming the option and the offending value.
   
   ### Tests
   
   - `RangeBitmapFileIndexTest#testChunkSizeBeyondIntRangeRejected`: `"2g"` 
rejected with a message containing `chunk-size`; `"16mb"` in-range still writes 
and serializes (RED verified on master: the test fails because the old code did 
not reject at writer creation).
   
   ### API and Format
   
   This is a new validation on an existing user option; values that previously 
worked are unaffected. Values that previously "worked" by accident (above 2GB) 
could never produce a working index — they crashed mid-write.
   
   ### Documentation
   
   None needed — the option behavior on out-of-range values was a crash, not a 
documented feature.


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

Reply via email to