Re: Review Request 49996: SENTRY-1399: Integrate Fencer with SentryStore

2016-07-15 Thread Colin McCabe
> On July 15, 2016, 12:17 a.m., Sravya Tirukkovalur wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/service/persistent/DelegateSentryStore.java, > > line 76 > > > > > >

Re: Review Request 49996: SENTRY-1399: Integrate Fencer with SentryStore

2016-07-14 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49996/#review142297 --- Fix it, then Ship it! Thanks Colin! +1 with minor comments.