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: issues-unsubscr...@iceberg.apache.org.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
