----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54798/ -----------------------------------------------------------
(Updated Jan. 17, 2017, 7:34 p.m.) Review request for sentry, Alexander Kolbasov, Hao Hao, Vamsee Yarlagadda, and Vadim Spector. Changes ------- Addressed review comments Repository: sentry Description ------- Added code changes to SentryStore to have proper message (where the exceptions are originated). If this is done we can avoid decorating them later. I saw issue with exceptions created while role create/drop and made required changes. Diffs (updated) ----- sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java b7ae63430cddb81cb3235a7b04ecf95410d8604f sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyStoreProcessor.java 898632ddef6ac5eec7b0e240c596f57f427fda9d Diff: https://reviews.apache.org/r/54798/diff/ Testing ------- I have performed tests using Sentrytool Client. Thanks, kalyan kumar kalvagadda
