huaxingao commented on PR #15975: URL: https://github.com/apache/iceberg/pull/15975#issuecomment-4364994632
Thanks @krisnaru, you're right. My PR only fixes the full-copy path. Since you already have a tested implementation that fixes both the full and incremental cases, I think it makes more sense for you to send the fix. Are you planning to open a PR for #14458? If yes, I'll close this one and review yours. If you'd rather not, let me know and I'll expand this PR to cover both cases. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
