JingsongLi commented on PR #3299: URL: https://github.com/apache/paimon/pull/3299#issuecomment-2109112146
> > Hi @hzjhjjyy , is this come from user requirement? I understand this optimization, but I'm hesitant to move forward with it. > > I think this is to improve the efficiency and scope of update. Naturally, I’m also willing to decide how to handle this pr based on your advice. Hi @hzjhjjyy for your inputs: 1. efficiency of update: In updates, the biggest consumption is twofold: first, discovering this data from the file; second, rewriting the file or using MOR technology. The optimization effect of some updates is not significant. 2. scope of update: I get this can support FieldLastValueAgg, but the default is FieldLastNonNullValueAgg. Considering these two points, and the changes made by this PR to the current topology are not very worthwhile. -- 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: issues-unsubscr...@paimon.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
