agnes-xinyi-lu opened a new issue, #6778: URL: https://github.com/apache/iceberg/issues/6778
### Apache Iceberg version 1.0.0 ### Query engine None ### Please describe the bug 🐞 Current [HttpClient](https://github.com/apache/iceberg/blob/master/core/src/main/java/org/apache/iceberg/rest/HTTPClient.java#:~:text=%7D%20catch%20(IOException%20e,%7D) in RestCatalog converts all the IOExceptions to RestException instead of CommitStateUnknownException, if exception happens after the server side commits successfully, client would treat it as a runtime exception and cleanup datafiles that actually got committed. A safer option might be to load table and check commit status when IOException happens, if we are still not able to get results due to network issues, maybe convert to CommitStateUnknownException? -- 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]
