Re: Review Request 51449: SENTRY-1422: JDO deadlocks while processing grant while a background thread processes Notificationlogs

2016-10-13 Thread Colin Ma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51449/ --- (Updated Oct. 14, 2016, 2:48 a.m.) Review request for sentry. Repository: sen

Re: Review Request 52795: SENTRY-1499: Add feature flag for using NotificationLog

2016-10-13 Thread Sravya Tirukkovalur
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52795/#review152586 --- Thanks for the patch Hao! I suspect new Listener tests would fail

Re: How does Sentry authorize SQL Insert

2016-10-13 Thread Lenni Kuff
Hi, That does sound like a potential bug. What privileges have been granted to the user (you can use SHOW CURRENT ROLES and SHOW GRANT ROLE to view the privileges). Can you also share the full INSERT statement? We should follow up with a JIRA if this is an issue. Thanks, Lenni On Thu, Oct 13, 201

How does Sentry authorize SQL Insert

2016-10-13 Thread An Qin
Hi, I developed a system with Sentry to authorize SQL users. When I execute an insert SQL (insert into ...), I notice that it fires 3 PreReadTableEvents, and then a PreAlterTableEvent to MetastoreAuthzBinding. Why is there a PreAlterTableEvent for this SQL? The user has no permission to insert