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]

Reply via email to