C-Loftus opened a new pull request, #1185: URL: https://github.com/apache/iceberg-go/pull/1185
Addresses https://github.com/apache/iceberg-go/issues/1096 - Removes interfaces for `MkdirIO` and `ReadDirIO` to prepare for hadoop cloud storage backends. - Refactors hadoop catalog to use `MkdirAll` and `WalkDir` in their place. - Cleans up the comments for those interfaces to reflect current usage. - Adds on new helper function `CheckedMkdirAll`, which before calling `MkdirAll`, walks the directory and ensures that each subdirectory of the full path exists. - Currently this is only used once since only one previous `Mkdir` call checked for errors if the directory already existed. That being said, I think it helps to abstract logic a bit and I'm thinking that it likely may be used in other parts of the Hadoop catalog. -- 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]
