Re: Review Request 64955: SENTRY-2109: Fix the logic of identifying HMS out of Sync

2018-01-18 Thread Vadim Spector via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64955/#review195758 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr

Re: Review Request 65224: SENTRY-2123: Specify Thrift source code path for Javadoc

2018-01-18 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65224/#review195755 --- Ship it! Ship It! - Arjun Mishra On Jan. 18, 2018, 6:18 p.m.

Re: Review Request 64820: SENTRY-2106: Remove sentry dependency on HMS table NOTIFICATION_SEQUENCE when checking if Sentry is out-of-sync with HMS

2018-01-18 Thread Vadim Spector via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64820/#review195750 --- Ship it! Ship It! - Vadim Spector On Jan. 16, 2018, 3:50 p.m

Re: Review Request 64820: SENTRY-2106: Remove sentry dependency on HMS table NOTIFICATION_SEQUENCE when checking if Sentry is out-of-sync with HMS

2018-01-18 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64820/#review195749 --- Ship it! Ship It! - Na Li On Jan. 16, 2018, 3:50 p.m., Arjun

Re: Review Request 65224: SENTRY-2123: Specify Thrift source code path for Javadoc

2018-01-18 Thread Brian Towles via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65224/#review195740 --- Ship it! Ship It! - Brian Towles On Jan. 18, 2018, 12:18 p.m

Review Request 65224: SENTRY-2123: Specify Thrift source code path for Javadoc

2018-01-18 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65224/ --- Review request for sentry, Arjun Mishra, Brian Towles, kalyan kumar kalvagadda,

Re: Review Request 65046: SENTRY-1819 HMSFollower and friends do not belong in sentry.service.thrift

2018-01-18 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65046/#review195726 --- Ship it! Ship It! - Sergio Pena On Jan. 11, 2018, 10:14 p.m.