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

   ### Purpose
   
   close #10918
   
   `DataEvolutionTableDataWrite` writes the updated columns of a data evolution 
table for `MERGE INTO` and `UPDATE`. It created an `IOManager` but never passed 
it to the write, so with `write-buffer-for-append = true` the first spill of 
the append write buffer failed with a `NullPointerException` in 
`ExternalBuffer.spill`.
   
   The write now gets the `IOManager` through `withIOManager`, like 
`PaimonDataWrite` and `PaimonV2DataWriter`.
   
   ### Tests
   
   `RowTrackingTestBase` "Data Evolution: merge into with a spilling append 
write buffer": a `MERGE INTO` on a data evolution table with 
`write-buffer-for-append` and a small `write-buffer-size`, so the buffer 
spills, then checks every updated row against its id. On master it fails with 
the `NullPointerException`.
   
   ### API and Format
   
   no
   
   ### Documentation
   
   no
   


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