----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4602/#review6608 -----------------------------------------------------------
Ship it! +1 - Arvind On 2012-04-01 09:19:53, Mike Percy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4602/ > ----------------------------------------------------------- > > (Updated 2012-04-01 09:19:53) > > > Review request for Flume. > > > Summary > ------- > > Client API compatibility with Flume 1.1.0 was broken with FLUME-962; This > patch restores backcompat. > > > This addresses bug FLUME-1087. > https://issues.apache.org/jira/browse/FLUME-1087 > > > Diffs > ----- > > flume-ng-sdk/src/main/java/org/apache/flume/api/RpcClientFactory.java > 25122a3 > flume-ng-sdk/src/test/java/org/apache/flume/api/TestRpcClientFactory.java > 65624b3 > > Diff: https://reviews.apache.org/r/4602/diff > > > Testing > ------- > > Added simple unit tests, which pass. > > > Thanks, > > Mike > >
