Re: Review Request 59174: SENTRY-1761 Add test for concurrent HMS followers processing notifications

2017-06-05 Thread kalyan kumar kalvagadda
> On June 6, 2017, 12:07 a.m., Vamsee Yarlagadda wrote: > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/TestConcurrentHMSFollowers.java > > Lines 47 (patched) > > > > > > IS this

Re: Review Request 59174: SENTRY-1761 Add test for concurrent HMS followers processing notifications

2017-06-05 Thread Vamsee Yarlagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59174/#review176968 --- sentry-provider/sentry-provider-db/src/test/java/org/apache/sentr

Re: Review Request 59174: SENTRY-1761 Add test for concurrent HMS followers processing notifications

2017-05-19 Thread Sergio Pena
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59174/#review175558 --- sentry-provider/sentry-provider-db/src/test/java/org/apache/sentr

Re: Review Request 59174: SENTRY-1761 Add test for concurrent HMS followers processing notifications

2017-05-18 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59174/ --- (Updated May 18, 2017, 7:16 p.m.) Review request for sentry, Alexander Kolbasov

Re: Review Request 59174: SENTRY-1761 Add test for concurrent HMS followers processing notifications

2017-05-18 Thread kalyan kumar kalvagadda
> On May 16, 2017, 6:54 p.m., Sergio Pena wrote: > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/TestConcurrentHMSFollowers.java > > Lines 47 (patched) > > > > > > Does this test

Re: Review Request 59174: SENTRY-1761 Add test for concurrent HMS followers processing notifications

2017-05-16 Thread Sergio Pena
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59174/#review175134 --- sentry-provider/sentry-provider-db/src/test/java/org/apache/sentr

Re: Review Request 59174: SENTRY-1761 Add test for concurrent HMS followers processing notifications

2017-05-15 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59174/#review175019 --- Ship it! Ship It! - Na Li On May 15, 2017, 4:30 p.m., kalyan

Re: Review Request 59174: SENTRY-1761 Add test for concurrent HMS followers processing notifications

2017-05-15 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59174/ --- (Updated May 15, 2017, 4:30 p.m.) Review request for sentry, Alexander Kolbasov

Re: Review Request 59174: SENTRY-1761 Add test for concurrent HMS followers processing notifications

2017-05-15 Thread kalyan kumar kalvagadda
> On May 15, 2017, 4:21 p.m., Sergio Pena wrote: > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/TestConcurrentHMSFollowers.java > > Lines 92 (patched) > > > > > > Need a space be

Re: Review Request 59174: SENTRY-1761 Add test for concurrent HMS followers processing notifications

2017-05-15 Thread Sergio Pena
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59174/#review174959 --- sentry-provider/sentry-provider-db/src/test/java/org/apache/sentr

Re: Review Request 59174: SENTRY-1761 Add test for concurrent HMS followers processing notifications

2017-05-12 Thread kalyan kumar kalvagadda
> On May 12, 2017, 3:28 p.m., Na Li wrote: > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/TestConcurrentHMSFollowers.java > > Lines 144 (patched) > > > > > > waiting for 5000 se

Re: Review Request 59174: SENTRY-1761 Add test for concurrent HMS followers processing notifications

2017-05-12 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59174/#review174798 --- sentry-provider/sentry-provider-db/src/test/java/org/apache/sentr

Review Request 59174: SENTRY-1761 Add test for concurrent HMS followers processing notifications

2017-05-11 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59174/ --- Review request for sentry, Alexander Kolbasov, Brian Towles, Hao Hao, Na Li, Ser