adiranjan25 commented on PR #18876: URL: https://github.com/apache/iceberg/pull/18876#issuecomment-6777670102
Hi Iceberg maintainers, I've submitted this PR to address #18872, where `rewrite_table_path` fails when equivalent local-file URI formats (`file:/` and `file:///`) are used across warehouse and metadata paths. The changes include: - Normalization of equivalent local-file URI representations during path rewriting. - Updated path-prefix validation. - Core regression tests covering both URI formats and invalid prefixes. - A Spark 4.1 integration regression verifying the rewritten file list and destination paths. The targeted core and Spark tests pass locally, along with Java formatting checks. The exact JDBC catalog configuration has not yet been independently tested. I would appreciate a maintainer's review of the approach, particularly the URI normalization and compatibility considerations. Thank you for your time and feedback! -- 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]
