----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58094/#review170792 -----------------------------------------------------------
sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/MetastoreCacheInitializer.java Line 301 (original), 301 (patched) <https://reviews.apache.org/r/58094/#comment243667> Why do we need synchronized here anympre as we are using thread safe vector? - Hao Hao On March 31, 2017, 4:21 a.m., Alexander Kolbasov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58094/ > ----------------------------------------------------------- > > (Updated March 31, 2017, 4:21 a.m.) > > > Review request for sentry, Hao Hao, kalyan kumar kalvagadda, Na Li, and Vadim > Spector. > > > Bugs: SENTRY-1683 > https://issues.apache.org/jira/browse/SENTRY-1683 > > > Repository: sentry > > > Description > ------- > > SENTRY-1683 MetastoreCacheInitializer has a race condition in handling > results list > > > Diffs > ----- > > > sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/MetastoreCacheInitializer.java > f9664f02d076f03a6481adbac24cefd72e90e152 > > > Diff: https://reviews.apache.org/r/58094/diff/1/ > > > Testing > ------- > > > Thanks, > > Alexander Kolbasov > >
