Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Flume >> mail # dev >> Review Request: FLUME-1896. Thrift Rpc Client


Copy link to this message
-
Re: Review Request: FLUME-1896. Thrift Rpc Client.

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

(Updated Feb. 7, 2013, 2:01 a.m.)
Review request for Flume.
Changes
-------

Added Factory methods, removed an old file which was Thrift generated.
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