----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8288/#review14110 -----------------------------------------------------------
This looks great. I assume it's just waiting for the patch to make the unit test pass? - Brock Noland On Nov. 30, 2012, 3:11 a.m., Mike Percy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8288/ > ----------------------------------------------------------- > > (Updated Nov. 30, 2012, 3:11 a.m.) > > > Review request for Flume. > > > Description > ------- > > Patch to fix exception handling in Load balancing RPC client > > > This addresses bug FLUME-1755. > https://issues.apache.org/jira/browse/FLUME-1755 > > > Diffs > ----- > > flume-ng-sdk/src/main/java/org/apache/flume/api/LoadBalancingRpcClient.java > 42297c1 > > flume-ng-sdk/src/test/java/org/apache/flume/api/TestLoadBalancingRpcClient.java > 49a69bf > > Diff: https://reviews.apache.org/r/8288/diff/ > > > Testing > ------- > > Added a unit test to exercise this case. Fails on current code base. > > > Thanks, > > Mike Percy > >
