Re: Review Request 58267: SENTRY-1629 sql changed needed for MAuthzPathsMapping.

2017-04-11 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58267/ --- (Updated April 11, 2017, 9:19 p.m.) Review request for sentry, Alexander

Re: Review Request 58267: SENTRY-1629 sql changed needed for MAuthzPathsMapping.

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

Re: Review Request 58267: SENTRY-1629 sql changed needed for MAuthzPathsMapping.

2017-04-10 Thread kalyan kumar kalvagadda
> On April 7, 2017, 9:43 p.m., Alexander Kolbasov wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MAuthzPathsMapping.java > > Line 31 (original), 32 (patched) > > >

Re: Review Request 58267: SENTRY-1629 sql changed needed for MAuthzPathsMapping.

2017-04-10 Thread kalyan kumar kalvagadda
> On April 10, 2017, 5:42 p.m., Na Li wrote: > > sentry-provider/sentry-provider-db/src/main/resources/007-SENTRY-1365.postgres.sql > > Lines 15 (patched) > > > > > > If we want to store unique "PATH_Name" only once,

Re: Review Request 58267: SENTRY-1629 sql changed needed for MAuthzPathsMapping.

2017-04-10 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58267/#review171451 ---

Re: Review Request 58267: SENTRY-1629 sql changed needed for MAuthzPathsMapping.

2017-04-07 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58267/#review171381 ---

Review Request 58267: SENTRY-1629 sql changed needed for MAuthzPathsMapping.

2017-04-07 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58267/ --- Review request for sentry, Alexander Kolbasov, Hao Hao, Na Li, Sergio Pena,