[GitHub] [incubator-hudi] lamber-ken edited a comment on issue #1128: [HUDI-453] Fix throw failed to archive commits error when writing data to MOR/COW table

[GitBox]

lamber-ken edited a comment on issue #1128: [HUDI-453] Fix throw failed to 
archive commits error when writing data to MOR/COW table
URL: https://github.com/apache/incubator-hudi/pull/1128#issuecomment-569461408
 
 
   Hi @bvaradar, I went through the process again. There are three kinds of 
empty files(`*.clean.requested`, `*.commit.requested`, 
`*.deltacommit.requested`).
   
   When reading `*.clean.requested` files, will throw `Not an Avro data file` 
error. Because `org.apache.avro.file.DataFileReader#openReader` cat not read it.
   
   When reading `*.commit.requested`, `*.deltacommit.requested` files, 
`HoodieCommitMetadata#fromBytes` checks whether empty commit file or not, 
return a new HoodieCommitMetadata instance when meets empty commit file.
   
   `HoodieActiveTimeline#saveToCleanRequested` created the empty 
`*.clean.requested` files.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services