Re: Review Request 47640: SENTRY-1260: ArrayIndexOutOfBoundsException in PathsUpdate.parsePath can cause MetastoreCacheInitializer startup to fail

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

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

[Vote] Merge SENTRY-1205 to master

2016-07-15 Thread Ma, Junjie
Hi all, All subtasks for SENTRY-1205 are resolved, and it can be merged to master. Please help to review the patch and vote. This vote will close no sooner than July 19, thanks. []+1 agree to merge []-1 don't merge, because .. best regards, Colin Ma