findinpath commented on code in PR #15806:
URL: https://github.com/apache/iceberg/pull/15806#discussion_r3004297099
##########
azure/src/main/java/org/apache/iceberg/azure/adlsv2/ADLSInputStream.java:
##########
@@ -184,6 +191,7 @@ private DataLakeFileOpenInputStreamResult
openRange(FileRange range) {
try {
return fileClient.openInputStream(getInputOptions(range));
} catch (RuntimeException e) {
+ throwNotFoundIfPresent(e, location);
Review Comment:
A file may be removed may happen also WHILE being read and we could have as
well situations in which NotFoundException should be thrown.
I didn't add this handling though in the `read(..)` methods yet because I
consider the scenario rather exotic (and the next retry would catch anyway the
fact that the file is not found).
--
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]
