kevinjqliu commented on PR #13786: URL: https://github.com/apache/iceberg/pull/13786#issuecomment-3688136100
> If it helps, the Spark 3.4 and Spark 3.5 implementations are (nearly) identical to Spark 4.0. If the review focuses only on Spark 4.0, I can make sure the same changes are made for the other Spark version branches. Generally I do find it easier to review the PR if we have 1 PR focused on a single Spark version, such as for Spark 4.0. Once we get that PR merged, we can add followup PR that just copy/paste (hopefully) the same code to other Spark versions. If you look through the repo, this is what we often do as "backport" PRs. Focusing on just the spark 4 integration might cut down the number of files to review by 1/3 :) -- 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]
