----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24961/ -----------------------------------------------------------
(Updated Sept. 4, 2014, 10:23 a.m.) Review request for sentry, Arun Suresh, Prasad Mujumdar, and Sravya Tirukkovalur. Changes ------- fix a bug of rename/drop privilege if this privilege is not exsit in db, but we should rename/drop the child privileges of it. Also add test cases to test it. Repository: sentry Description ------- rename table should get child privileges firstly, then add new child privileges with new tablename Diffs (updated) ----- sentry-core/sentry-core-model-db/src/main/java/org/apache/sentry/core/model/db/DBModelAuthorizable.java de35bfa sentry-provider/sentry-provider-db/src/gen/thrift/gen-javabean/org/apache/sentry/provider/db/service/thrift/TSentryAuthorizable.java 59418a3 sentry-provider/sentry-provider-db/src/gen/thrift/gen-javabean/org/apache/sentry/provider/db/service/thrift/TSentryPrivilege.java 54b6204 sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java 33600e9 sentry-provider/sentry-provider-db/src/main/resources/sentry_policy_service.thrift b14616b sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryPrivilege.java 91d3171 sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java 985a73d Diff: https://reviews.apache.org/r/24961/diff/ Testing ------- test case are included. Thanks, Xiaomeng Huang
