Re: Review Request 57220: SENTRY-1638. Update MSentryPermChange table to add a column for notification ID

2017-03-02 Thread kalyan kumar kalvagadda
> On March 2, 2017, 5:40 p.m., kalyan kumar kalvagadda wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/package.jdo > > Lines 287 (patched) > > > > > > SENTRY-16

Re: Review Request 57220: SENTRY-1638. Update MSentryPermChange table to add a column for notification ID

2017-03-02 Thread Lei Xu
> On March 2, 2017, 9:40 a.m., kalyan kumar kalvagadda wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/package.jdo > > Lines 287 (patched) > > > > > > SENTRY-16

Re: Review Request 57220: SENTRY-1638. Update MSentryPermChange table to add a column for notification ID

2017-03-02 Thread kalyan kumar kalvagadda
> On March 2, 2017, 5:40 p.m., kalyan kumar kalvagadda wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/package.jdo > > Lines 287 (patched) > > > > > > SENTRY-16

Re: Review Request 57220: SENTRY-1638. Update MSentryPermChange table to add a column for notification ID

2017-03-02 Thread Lei Xu
> On March 2, 2017, 9:40 a.m., kalyan kumar kalvagadda wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/package.jdo > > Lines 287 (patched) > > > > > > SENTRY-16

Re: Review Request 57220: SENTRY-1638. Update MSentryPermChange table to add a column for notification ID

2017-03-02 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57220/#review167691 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr

Re: Review Request 57220: SENTRY-1638. Update MSentryPermChange table to add a column for notification ID

2017-03-01 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57220/#review167595 --- Ship it! Ship It! - Alexander Kolbasov On March 1, 2017, 10:

Re: Review Request 57220: SENTRY-1638. Update MSentryPermChange table to add a column for notification ID

2017-03-01 Thread Lei Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57220/ --- (Updated March 1, 2017, 2:23 p.m.) Review request for sentry and Hao Hao. Cha

Re: Review Request 57220: SENTRY-1638. Update MSentryPermChange table to add a column for notification ID

2017-03-01 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57220/#review167584 --- Fix it, then Ship it! Ship It! sentry-provider/sentry-provide

Review Request 57220: SENTRY-1638. Update MSentryPermChange table to add a column for notification ID

2017-03-01 Thread Lei Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57220/ --- Review request for sentry and Hao Hao. Bugs: SENTRY-1638 https://issues.apa