----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54454/ -----------------------------------------------------------
(Updated Dec. 19, 2016, 11:25 p.m.) Review request for sentry, Alexander Kolbasov, Hao Hao, Vamsee Yarlagadda, and Vadim Spector. Changes ------- I'm working on SENTRY-1547 and SENTRY-1548. Fixes for both of them are in place. Last patch was bit confusing as weel as I had to remove the changes done for SENTRY-1547. This diff has changes for both of them. Repository: sentry Description ------- SENTRY-1548 Setting GrantOption to UNSET upsets Sentry I have made changes assuming that grant option is either true/false removing unset. Also, added code so that sentry server could validate the TSentryPrivilege object constructed from the Thrift message received client. If the validation is failed exception is raised and appropriate error is message is sent. Diffs (updated) ----- sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyStoreProcessor.java 898632d Diff: https://reviews.apache.org/r/54454/diff/ Testing ------- Verfied the changes using sentry thrift client. Thanks, kalyan kumar kalvagadda