Jordano-Dremio commented on code in PR #12755:
URL: https://github.com/apache/iceberg/pull/12755#discussion_r2036203729
##########
core/src/main/java/org/apache/iceberg/ManifestListWriter.java:
##########
@@ -136,7 +136,7 @@ protected FileAppender<ManifestFile> newAppender(OutputFile
file, Map<String, St
.build();
} catch (IOException e) {
- throw new RuntimeIOException(e, "Failed to create snapshot list writer
for path: %s", file);
+ throw new RuntimeIOException(e, "Failed to create snapshot list writer
for path: %s", file.location());
Review Comment:
Hi @nastra, Thank you for taking a quick scan. It appears I've had a syncing
issue thats resulting in why I'm missing a few places. I will ping you once
this is fully resolved and ready for review.
All the best
--
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: issues-unsubscr...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
