Re: Review Request 64451: SENTRY-2094: Enable TestHDFSIntegrationWithHA tests

2017-12-08 Thread Sergio Pena via Review Board
> On Dec. 8, 2017, 8:14 p.m., Na Li wrote: > > You can remove some comment no longer valid such as "//TODO: SENTRY-795: > > HDFS permissions do not sync when Sentry restarts in HA mode." True. I'll do that. - Sergio --- This is an

Re: Review Request 64451: SENTRY-2094: Enable TestHDFSIntegrationWithHA tests

2017-12-08 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64451/#review193276 --- You can remove some comment no longer valid such as "//TODO:

Re: Review Request 64451: SENTRY-2094: Enable TestHDFSIntegrationWithHA tests

2017-12-08 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64451/#review193269 --- Ship it! Ship It! - Na Li On Dec. 8, 2017, 3:53 p.m.,

Review Request 64451: SENTRY-2094: Enable TestHDFSIntegrationWithHA tests

2017-12-08 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64451/ --- Review request for sentry, Alexander Kolbasov, kalyan kumar kalvagadda, and Na