kbendick opened a new pull request, #5205: URL: https://github.com/apache/iceberg/pull/5205
When cleaning up some error prone warnings, I noticed that the `ExceptionUtil.runSafely` methods are not used anywhere other than from their own tests. Given that we are going to have API stability guarantees come 1.0, we should consider deprecating these for 0.14.0 and then removing them at 1.0 so we're not giving API / ABI compatibility guarantees for code that isn't used within the library and gives error prone warnings that are difficult to reason about. Original PR where the discussion of these functions took place: https://github.com/apache/iceberg/pull/5190 -- 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]
