Re: Review Request 52916: SENTRY-1505 CommitContext isn't used by anything and should be removed

2016-11-05 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52916/#review155069 --- Ship it! Ship It! - Hao Hao On Nov. 3, 2016, 3:58 a.m.,

Re: Review Request 52916: SENTRY-1505 CommitContext isn't used by anything and should be removed

2016-11-03 Thread Li Li
> On Nov. 3, 2016, 9:59 p.m., Hao Hao wrote: > > sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/generic/service/persistent/DelegateSentryStore.java, > > line 88 > > > > > > Make it

Re: Review Request 52916: SENTRY-1505 CommitContext isn't used by anything and should be removed

2016-11-03 Thread Alexander Kolbasov
> On Nov. 3, 2016, 9:59 p.m., Hao Hao wrote: > > sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/generic/service/persistent/DelegateSentryStore.java, > > line 88 > > > > > > Make it

Re: Review Request 52916: SENTRY-1505 CommitContext isn't used by anything and should be removed

2016-11-03 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52916/#review154821 ---

Re: Review Request 52916: SENTRY-1505 CommitContext isn't used by anything and should be removed

2016-11-03 Thread Li Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52916/#review154809 --- LGTM. fix it and ship it

Re: Review Request 52916: SENTRY-1505 CommitContext isn't used by anything and should be removed

2016-11-03 Thread Alexander Kolbasov
> On Nov. 3, 2016, 7:02 a.m., Colin Ma wrote: > > sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/generic/service/persistent/SentryStoreLayer.java, > > line 39 > > > > > > Do we

Re: Review Request 52916: SENTRY-1505 CommitContext isn't used by anything and should be removed

2016-11-03 Thread Colin Ma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52916/#review154680 ---

Re: Review Request 52916: SENTRY-1505 CommitContext isn't used by anything and should be removed

2016-11-02 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52916/ --- (Updated Nov. 3, 2016, 3:58 a.m.) Review request for sentry, Colm O