----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54798/#review160921 -----------------------------------------------------------
agreed with Vamsee, it needs a trimmedNamed + "_Already Exists" space before all messages - Mat Crocker On Dec. 16, 2016, 12:23 a.m., kalyan kumar kalvagadda wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54798/ > ----------------------------------------------------------- > > (Updated Dec. 16, 2016, 12:23 a.m.) > > > Review request for sentry, Alexander Kolbasov, Hao Hao, Vamsee Yarlagadda, > and Vadim Spector. > > > 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 > ----- > > > sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java > 742798d > > sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyStoreProcessor.java > 898632d > > Diff: https://reviews.apache.org/r/54798/diff/ > > > Testing > ------- > > I have performed tests using Sentrytool Client. > > > Thanks, > > kalyan kumar kalvagadda > >
