[GitHub] [hudi] satishkotha commented on pull request #2422: [HUDI-1276] [HUDI-1459] Make Clustering/ReplaceCommit and Metadata table be compatible

[GitBox <gi...@apache.org>]

satishkotha commented on pull request #2422:
URL: https://github.com/apache/hudi/pull/2422#issuecomment-757383826


   > @satishkotha This looks good and also much cleaner.
   > 
   > Only thing to do IMO is to make sure we don't throw an error when archival fails to delete the replaced file groups later.
   > 
   > ```
   >  boolean deleteSuccess = deleteReplacedFileGroups(context, hoodieInstant);
   >         if (!deleteSuccess) {
   >           // throw error and stop archival if deleting replaced file groups failed.
   >           throw new HoodieCommitException("Unable to delete file(s) for " + hoodieInstant.getFileName());
   >         }
   > ```
   > 
   > I think we can can that code, if you are comfortable with this. Or just `log.WARN` instead of exception for now, with a follow on 0.8.0 JIRA to do that.
   
   @vinothchandar Changed to log.warn and created HUDI-1518.


----------------------------------------------------------------
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:
users@infra.apache.org