Re: FAQs?

2016-04-21 Thread Anne Yu
Hi Bhooshan, Thanks for reporting this issue. If you have clear idea of what's going on, since you've spent time debugging the code, please feel free to create a jira (https://issues.apache.org/jira/browse/SENTRY/) then post a fix. Committers will be very happy to do the code review. Best, Anne

Re: FAQs?

2016-04-21 Thread Bhooshan Mogal
Thanks Anne. Like I said, I'm pretty sure what's going on. But it seems very unlikely that we could add a fix for this in Sentry, since the error occurs in the thrift communication between SentryService and SentryGenericServiceClientDefultImpl. To me, this seemed like a good candidate for some FAQs

Re: FAQs?

2016-04-21 Thread Anne Yu
Hi Bhooshan, IC, FAQ session could be very helpful. We have a upstream space for all sentry docs (https://cwiki.apache.org/confluence/display/SENTRY/Home). You might want to add a dedicated page for FAQ. Besides if "sentry.service.security.mode" is inconsistent between server and client, the clie

Re: FAQs?

2016-04-21 Thread Bhooshan Mogal
Thanks. Yes, it would be helpful to throw an appropriate error message. The problem though is we have to presume that the service and the client use different sentry-site.xmls. In the service process, you do not have access to the client sentry-site.xml and vice versa. So I don't think there is a

Re: FAQs?

2016-04-21 Thread Bhooshan Mogal
Never mind. I created an account on Confluence. I'll add the page later. On Thu, Apr 21, 2016 at 2:54 PM, Bhooshan Mogal wrote: > Thanks. Yes, it would be helpful to throw an appropriate error message. > The problem though is we have to presume that the service and the client > use different sen

Review Request 46542: SENTRY-583: Add boundary condition test coverage to HDFS synchronization test suite around max #of groups

2016-04-21 Thread Anne Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46542/ --- Review request for sentry, Hao Hao and Lenni Kuff. Bugs: SENTRY-583 https:/

Re: Review Request 46542: SENTRY-583: Add boundary condition test coverage to HDFS synchronization test suite around max #of groups

2016-04-21 Thread Anne Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46542/ --- (Updated April 21, 2016, 11:47 p.m.) Review request for sentry, Hao Hao and Len

Review Request 46548: SENTRY-1212: Small authorization and compatibility checking bugs in Sentry conversion tool

2016-04-21 Thread Gregory Chanan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46548/ --- Review request for sentry, Sravya Tirukkovalur and Vamsee Yarlagadda. Repositor