-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/9284/
-----------------------------------------------------------
(Updated Feb. 7, 2013, 3:31 a.m.)
Review request for Flume.
Changes
-------
Use TFastFramedTransport and TCompactProtocol.
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
