You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "findinpath (via GitHub)" <gi...@apache.org> on 2026/03/28 05:41:31 UTC
[PR] ADLS: Throw NotFoundException for inexistent input file [iceberg]
findinpath opened a new pull request, #15806:
URL: https://github.com/apache/iceberg/pull/15806
Signal to the TableOperations that there is no retry needed for files which do not exist.
Relates to https://github.com/apache/iceberg/pull/15734
Fixes https://github.com/apache/iceberg/issues/15760
--
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-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] ADLS: Throw NotFoundException for inexistent input file [iceberg]
Posted by "findinpath (via GitHub)" <gi...@apache.org>.
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] ADLS: Throw NotFoundException for inexistent input file [iceberg]
Posted by "nastra (via GitHub)" <gi...@apache.org>.
nastra merged PR #15806:
URL: https://github.com/apache/iceberg/pull/15806
--
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-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org