----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34237/#review83863 -----------------------------------------------------------
Ship it! Ship It! - Alok Lal On May 14, 2015, 3:29 p.m., Ramesh Mani wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34237/ > ----------------------------------------------------------- > > (Updated May 14, 2015, 3:29 p.m.) > > > Review request for ranger, Alok Lal, dilli dorai, Abhay Kulkarni, and Madhan > Neethiraj. > > > Repository: ranger > > > Description > ------- > > RANGER-476:ServiceName should be used in Lookup Connection cache in > Connection Manager instead of ServiceType as we can have multiple Services > for same service type > > > Diffs > ----- > > > hbase-agent/src/main/java/org/apache/ranger/services/hbase/client/HBaseConnectionMgr.java > 5c1c73b > > hbase-agent/src/main/java/org/apache/ranger/services/hbase/client/HBaseResourceMgr.java > 4ce6a8d > > hdfs-agent/src/main/java/org/apache/ranger/services/hdfs/client/HdfsConnectionMgr.java > d62bb9c > > hdfs-agent/src/main/java/org/apache/ranger/services/hdfs/client/HdfsResourceMgr.java > 9161a5a > > knox-agent/src/main/java/org/apache/ranger/services/knox/client/KnoxResourceMgr.java > e0206e7 > > plugin-kms/src/main/java/org/apache/ranger/services/kms/client/KMSResourceMgr.java > 94ca822 > > plugin-yarn/src/main/java/org/apache/ranger/services/yarn/client/YarnResourceMgr.java > 70a6dfb > > storm-agent/src/main/java/org/apache/ranger/services/storm/client/StormResourceMgr.java > c572898 > > Diff: https://reviews.apache.org/r/34237/diff/ > > > Testing > ------- > > - serviceName instead of Servicetype used in Connection cache > - Added synchornized block for all the lookup client > > > Thanks, > > Ramesh Mani > >
