Re: Review Request 64452: SENTRY-2091: User-based Privilege is broken by SENTRY-769

2018-01-29 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64452/#review196465 --- sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/se

Re: Review Request 64452: SENTRY-2091: User-based Privilege is broken by SENTRY-769

2018-01-29 Thread Steve Moist via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64452/#review196462 --- sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/se

Re: Review Request 64452: SENTRY-2091: User-based Privilege is broken by SENTRY-769

2018-01-29 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64452/#review196435 --- sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/se

Re: Review Request 64452: SENTRY-2091: User-based Privilege is broken by SENTRY-769

2018-01-22 Thread Na Li via Review Board
> On Dec. 15, 2017, 10:48 p.m., Vadim Spector wrote: > > sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestUserManagement.java > > Line 372 (original), 372 (patched) > > > > > > Are

Re: Review Request 64452: SENTRY-2091: User-based Privilege is broken by SENTRY-769

2018-01-22 Thread Na Li via Review Board
> On Dec. 19, 2017, 9:58 p.m., Steve Moist wrote: > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestGrantUserToRole.java > > Lines 280 (patched) > > > > > > Worth it to add @

Re: Review Request 64452: SENTRY-2091: User-based Privilege is broken by SENTRY-769

2018-01-22 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64452/ --- (Updated Jan. 22, 2018, 9:16 p.m.) Review request for sentry, Arjun Mishra, kal

Re: Review Request 64452: SENTRY-2091: User-based Privilege is broken by SENTRY-769

2017-12-19 Thread Steve Moist via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64452/#review194184 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr

Re: Review Request 64452: SENTRY-2091: User-based Privilege is broken by SENTRY-769

2017-12-15 Thread Vadim Spector via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64452/#review193992 --- sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry

Re: Review Request 64452: SENTRY-2091: User-based Privilege is broken by SENTRY-769

2017-12-15 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64452/#review193979 --- Ship it! Ship It! - Arjun Mishra On Dec. 14, 2017, 8:54 p.m.

Re: Review Request 64452: SENTRY-2091: User-based Privilege is broken by SENTRY-769

2017-12-15 Thread Na Li via Review Board
> On Dec. 15, 2017, 3:37 p.m., Arjun Mishra wrote: > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestGrantUserToRole.java > > Lines 282 (patched) > > > > > > Is the fail stat

Re: Review Request 64452: SENTRY-2091: User-based Privilege is broken by SENTRY-769

2017-12-15 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64452/#review193920 --- sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/te

Re: Review Request 64452: SENTRY-2091: User-based Privilege is broken by SENTRY-769

2017-12-14 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64452/ --- (Updated Dec. 14, 2017, 8:54 p.m.) Review request for sentry, Arjun Mishra, kal

Re: Review Request 64452: SENTRY-2091: User-based Privilege is broken by SENTRY-769

2017-12-13 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64452/#review193725 --- sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/se

Review Request 64452: SENTRY-2091: User-based Privilege is broken by SENTRY-769

2017-12-13 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64452/ --- Review request for sentry, Arjun Mishra, kalyan kumar kalvagadda, Sergio Pena, a