Re: Review Request 65768: SENTRY-2144: Table Rename should allow database name change

2018-05-18 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65768/ --- (Updated May 18, 2018, 8:03 p.m.) Review request for sentry, Alexander Kolbasov

Re: Review Request 65768: SENTRY-2144: Table Rename should allow database name change

2018-04-10 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65768/#review200840 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr

Re: Review Request 65768: SENTRY-2144: Table Rename should allow database name change

2018-04-10 Thread Na Li via Review Board
> On March 14, 2018, 6:33 p.m., kalyan kumar kalvagadda wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/FullUpdateModifier.java > > Line 363 (original), 363 (patched) > > > >

Re: Review Request 65768: SENTRY-2144: Table Rename should allow database name change

2018-04-10 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65768/ --- (Updated April 10, 2018, 7:22 p.m.) Review request for sentry, Alexander Kolbas

Re: Review Request 65768: SENTRY-2144: Table Rename should allow database name change

2018-04-10 Thread kalyan kumar kalvagadda via Review Board
> On March 14, 2018, 6:33 p.m., kalyan kumar kalvagadda wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/FullUpdateModifier.java > > Line 363 (original), 363 (patched) > > > >

Re: Review Request 65768: SENTRY-2144: Table Rename should allow database name change

2018-03-30 Thread kalyan kumar kalvagadda via Review Board
> On March 14, 2018, 6:33 p.m., kalyan kumar kalvagadda wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/FullUpdateModifier.java > > Line 363 (original), 363 (patched) > > > >

Re: Review Request 65768: SENTRY-2144: Table Rename should allow database name change

2018-03-14 Thread Na Li via Review Board
> On March 14, 2018, 6:33 p.m., kalyan kumar kalvagadda wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/FullUpdateModifier.java > > Line 363 (original), 363 (patched) > > > >

Re: Review Request 65768: SENTRY-2144: Table Rename should allow database name change

2018-03-14 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65768/#review199197 --- Lina, Don't we need changes to notifcation processor to handle thi

Re: Review Request 65768: SENTRY-2144: Table Rename should allow database name change

2018-03-07 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65768/ --- (Updated March 7, 2018, 10:39 p.m.) Review request for sentry, Alexander Kolbas

Re: Review Request 65768: SENTRY-2144: Table Rename should allow database name change

2018-03-07 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65768/ --- (Updated March 7, 2018, 7:16 p.m.) Review request for sentry, Alexander Kolbaso