majian1998 opened a new pull request, #6928:
URL: https://github.com/apache/paimon/pull/6928
<!-- Please specify the module before the PR name: [core] ... or [flink] ...
-->
### Purpose
This PR removes some redundant catch (RuntimeException e) { throw e; }
blocks and simplifies exception handling.
In FilesystemCatalog, uncheck(...) already wraps exceptions (including
RuntimeException), and runWithLock(...) is not expected to throw meaningful
RuntimeException subtypes that need to be preserved, so the extra catch
(RuntimeException) branches were unnecessary and removed.
HiveCatalog is left unchanged since it can still throw business/validation
RuntimeException subtypes directly, where preserving the original type/message
is useful.
Similar cleanups were applied in other places with the same pattern to
reduce boilerplate and improve readability.
### Tests
<!-- List UT and IT cases to verify this change -->
### API and Format
<!-- Does this change affect API or storage format -->
### Documentation
<!-- Does this change introduce a new feature -->
--
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]