wombatu-kun commented on PR #16999: URL: https://github.com/apache/iceberg/pull/16999#issuecomment-4841565495
Welcome, and thanks for tackling this, @CoderNasim2023! This bug is already being fixed in #16498 (open since May 21), with the same approach - normalizing `location` to directory form (`stripTrailingSlash(location) + "/"`) before the `startsWith` filter, across all three Spark trees. #16498 additionally clarifies the `DeleteOrphanFiles.location` javadoc and adds a dedicated test that also covers the path-equals-location edge case, and it already has a review from @sungwy. To avoid splitting the effort, would you be up for reviewing or testing #16498 instead? That would help it land. I'll leave it to the maintainers to decide whether to close this one as a duplicate. -- 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]
