-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/9284/
-----------------------------------------------------------

(Updated Feb. 8, 2013, 10:40 p.m.)


Review request for Flume.


Changes
-------

Making sure the incoming list is not changed.


Description
-------

Added thrift rpc client. Detailed description posted on jira.


This addresses bug FLUME-1896.
    https://issues.apache.org/jira/browse/FLUME-1896


Diffs (updated)
-----

  flume-ng-legacy-sources/flume-thrift-source/pom.xml b9667cd 
  
flume-ng-legacy-sources/flume-thrift-source/src/main/java/com/cloudera/flume/handlers/thrift/EventStatus.java
 327107a 
  
flume-ng-legacy-sources/flume-thrift-source/src/main/java/com/cloudera/flume/handlers/thrift/Priority.java
 d2495d2 
  
flume-ng-legacy-sources/flume-thrift-source/src/main/java/com/cloudera/flume/handlers/thrift/ThriftFlumeEvent.java
 2bb6cfd 
  
flume-ng-legacy-sources/flume-thrift-source/src/main/java/com/cloudera/flume/handlers/thrift/ThriftFlumeEventServer.java
 0f2ad2d 
  flume-ng-legacy-sources/flume-thrift-source/src/main/thrift/aslv2 
PRE-CREATION 
  flume-ng-sdk/pom.xml ab066d5 
  flume-ng-sdk/src/main/java/org/apache/flume/api/HostInfo.java 7388a45 
  
flume-ng-sdk/src/main/java/org/apache/flume/api/RpcClientConfigurationConstants.java
 ab4c3de 
  flume-ng-sdk/src/main/java/org/apache/flume/api/RpcClientFactory.java 3c93921 
  flume-ng-sdk/src/main/java/org/apache/flume/api/ThriftRpcClient.java 
PRE-CREATION 
  flume-ng-sdk/src/main/java/org/apache/flume/thrift/Status.java PRE-CREATION 
  flume-ng-sdk/src/main/java/org/apache/flume/thrift/ThriftFlumeEvent.java 
PRE-CREATION 
  flume-ng-sdk/src/main/java/org/apache/flume/thrift/ThriftSourceProtocol.java 
PRE-CREATION 
  flume-ng-sdk/src/main/thrift/aslv2 PRE-CREATION 
  flume-ng-sdk/src/main/thrift/flume.thrift PRE-CREATION 
  flume-ng-sdk/src/test/java/org/apache/flume/api/TestThriftRpcClient.java 
PRE-CREATION 
  flume-ng-sdk/src/test/java/org/apache/flume/api/ThriftTestingSource.java 
PRE-CREATION 
  flume-ng-sources/flume-scribe-source/pom.xml 588207d 
  
flume-ng-sources/flume-scribe-source/src/main/java/org/apache/flume/source/scribe/LogEntry.java
 401a7e4 
  
flume-ng-sources/flume-scribe-source/src/main/java/org/apache/flume/source/scribe/ResultCode.java
 6bfa84c 
  
flume-ng-sources/flume-scribe-source/src/main/java/org/apache/flume/source/scribe/Scribe.java
 ccde51b 
  
flume-ng-sources/flume-scribe-source/src/main/java/org/apache/flume/source/scribe/ScribeSource.java
 6190b11 
  flume-ng-sources/flume-scribe-source/src/main/thrift/aslv2 PRE-CREATION 
  flume-ng-sources/flume-scribe-source/src/main/thrift/scribe-source.thrift 
PRE-CREATION 
  pom.xml faa67f9 

Diff: https://reviews.apache.org/r/9284/diff/


Testing
-------

Added unit tests for new code.


Thanks,

Hari Shreedharan

Reply via email to