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