C-Loftus commented on PR #1280: URL: https://github.com/apache/iceberg-go/pull/1280#issuecomment-4789880872
Thanks for your review @laskoviymishka ! I will make those changes. > make the remote path real — either extend blobFileIO to satisfy HadoopCatalogFS (with copy+delete Rename), or explicitly scope this PR as a prep step in the description To clarify I mainly wanted to get in a fix for the localfs issue. As such this PR is just the next step for https://github.com/apache/iceberg-go/issues/1096 and not the full implementation. I have edited the PR description to make that more explicit. I will save that logic for a follow up so it does not block iterations. -- 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]
