Chen, Junjie
Chen, Junjie
@garawalid thanks for the patch, it looks good to me except one small nit for code style.
@garawalid ， would you please update the pull request against the master?
@rdblue @aokolnychyi @flyrain Could you please help to take a look? This just includes the first part metadata change, will continue the writer changes if this is the right direction...
@flyrain Some delete files can be removed when committing, the related logic in `MergingSnapshotProducer#apply` compute the minimum sequence number of the data manifest and use it to filter out delete...
Also cc @jackye1995 @stevenzwu. Just found an opened issue as well #3789
@flyrain Just reread your idea, one question is how do you handle the #seq if the writer is an upsert writer? The #seq is unknown when committing.
@aokolnychyi Could you help to take a look?
Thanks for ping me, I will take a look this week.
Really nice PR, thanks @szehon-ho and @aokolnychyi for the effort! When can we merge this? I think it is ready and has been two months since the last review, which...
@szehon-ho，@rdblue Any update here?