danielcweeks commented on code in PR #6887:
URL: https://github.com/apache/iceberg/pull/6887#discussion_r1114853791
##########
api/src/main/java/org/apache/iceberg/Files.java:
##########
@@ -57,7 +57,7 @@ public PositionOutputStream create() {
}
if (!file.getParentFile().isDirectory() &&
!file.getParentFile().mkdirs()) {
Review Comment:
It looks like there are only four places not in test where it's being used
and two or three of those are in catch blocks. I think we could probably find
a better set of exceptions to use in the few places where it is thrown and
remove this entirely.
--
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]
