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

Switch to Threaded View
Flume, mail # dev - Review Request: Thrift compatibility issue with hbase-0.92


Copy link to this message
-
Review Request: Thrift compatibility issue with hbase-0.92
Hari Shreedharan 2013-04-07, 05:54

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

Review request for Flume.
Description
-------

Modified the ThriftSource to use a server available in Thrift 0.7, changed the pom to use the thrift 0.7 version. Regenerated all thrift sources.
This addresses bug FLUME-1974.
    https://issues.apache.org/jira/browse/FLUME-1974
Diffs
-----

  flume-ng-core/src/main/java/org/apache/flume/source/ThriftSource.java 979fd35
  flume-ng-legacy-sources/flume-thrift-source/src/main/java/com/cloudera/flume/handlers/thrift/EventStatus.java 549ea16
  flume-ng-legacy-sources/flume-thrift-source/src/main/java/com/cloudera/flume/handlers/thrift/Priority.java 6879817
  flume-ng-legacy-sources/flume-thrift-source/src/main/java/com/cloudera/flume/handlers/thrift/ThriftFlumeEvent.java 418c376
  flume-ng-legacy-sources/flume-thrift-source/src/main/java/com/cloudera/flume/handlers/thrift/ThriftFlumeEventServer.java 10f35c7
  flume-ng-sdk/src/main/java/org/apache/flume/thrift/Status.java c05f4e3
  flume-ng-sdk/src/main/java/org/apache/flume/thrift/ThriftFlumeEvent.java b8af99c
  flume-ng-sdk/src/main/java/org/apache/flume/thrift/ThriftSourceProtocol.java b041895
  flume-ng-sources/flume-scribe-source/src/main/java/org/apache/flume/source/scribe/LogEntry.java 792e196
  flume-ng-sources/flume-scribe-source/src/main/java/org/apache/flume/source/scribe/ResultCode.java 2ab0bcc
  flume-ng-sources/flume-scribe-source/src/main/java/org/apache/flume/source/scribe/Scribe.java 99f302f
  pom.xml 1388e86

Diff: https://reviews.apache.org/r/10326/diff/
Testing
-------

All unit tests pass.
Thanks,

Hari Shreedharan