zhuanshenbsj1 commented on PR #7405: URL: https://github.com/apache/hudi/pull/7405#issuecomment-1365704266
> [5341.patch.zip](https://github.com/apache/hudi/files/10305723/5341.patch.zip) Hi, @zhuanshenbsj1 , i have tried to review and applied a patch, add a weak guarantee on the archiver pre condition check, do you think that is okey ? It would be costy of we resolve all the clean metadata to see whether the clustering replaced files are all been cleaned. > > We can go ahead if you think it is okey, and it would be great if you can test it also. But this patch still not consider clustering completed time ? If previous clustering instant in timeline compelted later than following clean instant cleaned cutoff timeļ¼ there are still risks. -- 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]
