Re: Review Request 55094: SENTRY-1532: Sentry Web UI isn't working

2017-01-10 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55094/ --- (Updated Jan. 11, 2017, 1:35 a.m.) Review request for sentry, Colm O

Re: Review Request 54729: SENTRY-1538: Create schema for storing HMS path change and Sentry permission change.

2017-01-10 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54729/ --- (Updated Jan. 11, 2017, 1:13 a.m.) Review request for sentry, Alexander

Sentry-jdk-1.7 - Build # 753 - Fixed

2017-01-10 Thread Apache Jenkins Server
The Apache Jenkins build system has built Sentry-jdk-1.7 (build #753) Status: Fixed Check console output at https://builds.apache.org/job/Sentry-jdk-1.7/753/ to view the results.

Re: Review Request 54729: SENTRY-1538: Create schema for storing HMS path change and Sentry permission change.

2017-01-10 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54729/ --- (Updated Jan. 10, 2017, 11:49 p.m.) Review request for sentry, Alexander

Re: Review Request 54947: SENTRY-1556 Simplify privilege cleaning

2017-01-10 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54947/#review161142 ---

Re: Review Request 54947: SENTRY-1556 Simplify privilege cleaning

2017-01-10 Thread Vamsee Yarlagadda
> On Jan. 5, 2017, 8:34 p.m., Vamsee Yarlagadda wrote: > > sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/model/MSentryGMPrivilege.java, > > line 504 > > > > > > Just

Re: Review Request 55092: SENTRY-1581: Provide Log4J metrics reporter

2017-01-10 Thread Vamsee Yarlagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55092/#review161122 --- Ship it!

Re: Review Request 54729: SENTRY-1538: Create schema for storing HMS path change and Sentry permission change.

2017-01-10 Thread Hao Hao
> On Jan. 10, 2017, 8:24 p.m., Alexander Kolbasov wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryPermChange.java, > > line 87 > > > > > > iWhat is

Re: Review Request 55092: SENTRY-1581: Provide Log4J metrics reporter

2017-01-10 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55092/#review161112 --- Ship it! Ship It! - Hao Hao On Dec. 29, 2016, 11:20 p.m.,

Re: Review Request 54729: SENTRY-1538: Create schema for storing HMS path change and Sentry permission change.

2017-01-10 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54729/#review161110 --- Fix it, then Ship it!