> On June 26, 2012, 8:08 p.m., Hari Shreedharan wrote: > > Ship It!
Please attach the patch to the jira. Thanks! - Hari ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5503/#review8636 ----------------------------------------------------------- On June 21, 2012, 10:51 p.m., Mubarak Seyed wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/5503/ > ----------------------------------------------------------- > > (Updated June 21, 2012, 10:51 p.m.) > > > Review request for Flume. > > > Description > ------- > > LoadBalancingRpcClient should catch exception for invalid RpcClient and > failover to valid one. I just moved > > RpcClient client = getClient(host); > > to try block. > > > This addresses bug FLUME-1306. > https://issues.apache.org/jira/browse/FLUME-1306 > > > Diffs > ----- > > > trunk/flume-ng-sdk/src/main/java/org/apache/flume/api/LoadBalancingRpcClient.java > 1352719 > > Diff: https://reviews.apache.org/r/5503/diff/ > > > Testing > ------- > > Yes. > > > Thanks, > > Mubarak Seyed > >
