----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68268/ -----------------------------------------------------------
(Updated Aug. 8, 2018, 10:36 p.m.) Review request for sentry, Arjun Mishra, kalyan kumar kalvagadda, and Na Li. Bugs: sentry-2157 https://issues.apache.org/jira/browse/sentry-2157 Repository: sentry Description ------- This patch logs owner privileges grants and revokes. Diffs (updated) ----- sentry-service/sentry-service-server/src/main/java/org/apache/sentry/api/service/thrift/SentryPolicyStoreProcessor.java 61f9168b1970144dbf0b7a7378f2d25e70f1761d sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/audit/SentryAuditLogger.java PRE-CREATION sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/log/entity/JsonLogEntityFactory.java 61becceac881443b02182e6ab1012add4c046499 sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/log/util/CommandUtil.java 6479a6055e8c7087f0e484080ec9d46a9c146212 sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/log/util/Constants.java 6e91f8b9ead009629d6bccd206122b2071e8fd64 sentry-service/sentry-service-server/src/test/java/org/apache/sentry/provider/db/log/entity/TestJsonLogEntityFactory.java 307f38eadb65bf12dc6225cfe43a5d590657d055 Diff: https://reviews.apache.org/r/68268/diff/3/ Changes: https://reviews.apache.org/r/68268/diff/2-3/ Testing ------- I run the patch in a cluster and the audit logs is displaying the correct messages. {"serviceName":"Sentry-Service","userName":"ubuntu","impersonator":"","ipAddress":"/127.0.0.1","operation":"GRANT_PRIVILEGE","eventTime":"1533738512795","operationText":"OWNER privilege granted to USER: sergio","allowed":"true","databaseName":"default","tableName":"t2","column":null,"resourcePath":null,"objectType":"PRINCIPAL"} Thanks, Sergio Pena