alessandro-nori commented on PR #14886: URL: https://github.com/apache/iceberg/pull/14886#issuecomment-3772452087
> Really sorry for the delay in following up on this, the change looks good to me though typically we like to just start out with the latest Spark version and then have a separate backport PR for the older versions, especially for newer additions of functionality. Could we just keep this initial PR for 4.1? Hi @amogh-jahagirdar , no worries. Regarding introducing this change only to 4.1, if I'm doing things correctly this is not possible because the ImmutableDeleteOrphanFiles.Result builder throws an exception if I don't set `orphanFilesCount` for the other Spark versions. -- 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]
