yunyezhang-work commented on PR #734:
URL: https://github.com/apache/ranger/pull/734#issuecomment-3547004047

   > Hi @yunyezhang-work , I had fixed one related JIRA: 
https://issues.apache.org/jira/browse/RANGER-3965 .
   > 
   > My observation: It's more about how code is handling the runtime failures. 
Here, exception is occurring but KMS logic is not re-throwing the exception.
   > 
   > I suggest to add handling in org.apache.ranger.kms.dao.BaseDao methods. 
Here instead of catching and logging, we should throw the RuntimeException to 
break the API call.
   > 
   > Similar fix I merged as part of above JIRA. Pls see if that resolves your 
issues This is planned to go into next release.
   
   OK. The above-mentioned modifications can solve this problem.


-- 
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