Re: [PR] Core, Spark: Fix copy plan to include live files with expired snapshot IDs in RewriteTablePath [iceberg]

["huaxingao (via GitHub)" <gi...@apache.org>]

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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org