dineshchitlangia opened a new pull request #115: URL: https://github.com/apache/incubator-ratis/pull/115
## What changes were proposed in this pull request? Handled Exception appropriately. 1. InterruptedException should not be ignored, either re-interrupt or re-throw them 2. Catch Exception instead of Throwable Occurrences in Native API were ignored as Native API will be removed via RATIS-757 Certain occurrences on Throwable were not replaced by Exception as they use the interfaces defined in ratis-common/src/main/java/org/apache/ratis/util/function package and those interfaces extend Throwable. There needs to be a broader discussion on why they extend throwable and if we really need to change it. ## What is the link to the Apache JIRA https://issues.apache.org/jira/browse/RATIS-950 ## How was this patch tested? clean build, sonar, checkstyle in local ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
