Jiayi-Wang-db opened a new pull request, #3351:
URL: https://github.com/apache/parquet-java/pull/3351

   ### Rationale for this change
   Inside the `InternalParquetRecordWriter::Close` finally block, we call close 
on parquetFileWriter, which may cause incomplete data to be flushed to the 
cloud if an exception is thrown during the close .
   
   ### What changes are included in this PR?
   
   Remove `parquetFileWriter.close` out of finally block and added a unit test.
   ### Are these changes tested?
   
   Yes.
   
   ### Are there any user-facing changes?
   
   Users wouldn't get incomplet parquet files because of torn writes.
   
    Closes #3350
   


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