ConeyLiu opened a new pull request, #7171:
URL: https://github.com/apache/iceberg/pull/7171

   We use the current PartitionSpec for IcebergStreamWriter, which is fixed and 
will not change after the job started. While the PartitionSpec for 
IcebergStreamWriter is refreshed with the table snapshot changing. This could 
fail the fink sink job when updating the partition spec. Because we use the 
wrong partition spec to write those DataFiles/DeleteFiles to ManifestFile.
   
   For example, we got the following error when updating the partition spec:
   
   <img width="940" alt="error" 
src="https://user-images.githubusercontent.com/12733256/226884127-cc8d6732-7cec-4a31-bfc1-0b2ccebc3780.png";>
   
   In this patch, we keep the same PartitionSpec between `IcebergStreamWriter` 
and `IcebergFilesCommitter`
   


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