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

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

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
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

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