vyommani commented on code in PR #650:
URL: https://github.com/apache/ranger/pull/650#discussion_r2316381125


##########
kms/src/main/java/org/apache/hadoop/crypto/key/RangerKeyStore.java:
##########
@@ -529,6 +529,7 @@ public void dbOperationStore(XXRangerKeyStore 
rangerKeyStore) {
             }
         } catch (Exception e) {
             logger.error("dbOperationStore({}) error", 
rangerKeyStore.getAlias(), e);
+            throw new RuntimeException("Error while storing object in the DB. 
" + e.getMessage());

Review Comment:
   Please throw the new RuntimeException with the message and cause as follows
          throw new RuntimeException("Error while storing object in the DB.", 
e);
   This will make sure that propagated exception will have original stack trace.



-- 
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: dev-unsubscr...@ranger.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to