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 adopt message from existing local Scribe


Copy link to this message
-
Re: Review Request: Flume adopt message from existing local Scribe

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/6089/#review9810
-----------------------------------------------------------
Hi, this is great... I was thinking I would need to make a new Thrift source for flume, but this is exactly what we need.

I'm testing it out now, and it seems to work fine.

At a glance, the code looks fine from what I can see, but it seems like the thrift generated code is included in the patch? I'll have a proper look at it asap(might not be until after the weekend though) if no-one else has.

- Juhani Connolly
On July 30, 2012, 2:49 a.m., Denny Ye wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/6089/
> -----------------------------------------------------------
>
> (Updated July 30, 2012, 2:49 a.m.)
>
>
> Review request for Flume and Hari Shreedharan.
>
>
> Description
> -------
>
> There may someone like me that want to replace central Scribe with Flume to adopt existing ingest system, using smooth changes for application user.
> Here is the ScribeSource put into legacy folder without deserializing.
>
>
> This addresses bug https://issues.apache.org/jira/browse/FLUME-1382.
>     https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/FLUME-1382
>
>
> Diffs
> -----
>
>   trunk/flume-ng-dist/pom.xml 1363210
>   trunk/flume-ng-sources/flume-scribe-source/pom.xml PRE-CREATION
>   trunk/flume-ng-sources/flume-scribe-source/src/main/java/org/apache/flume/source/scribe/LogEntry.java PRE-CREATION
>   trunk/flume-ng-sources/flume-scribe-source/src/main/java/org/apache/flume/source/scribe/ResultCode.java PRE-CREATION
>   trunk/flume-ng-sources/flume-scribe-source/src/main/java/org/apache/flume/source/scribe/Scribe.java PRE-CREATION
>   trunk/flume-ng-sources/flume-scribe-source/src/main/java/org/apache/flume/source/scribe/ScribeSource.java PRE-CREATION
>   trunk/flume-ng-sources/pom.xml PRE-CREATION
>   trunk/pom.xml 1363210
>
> Diff: https://reviews.apache.org/r/6089/diff/
>
>
> Testing
> -------
>
> I already used ScribeSource into local environment and tested in past week. It can use the existing local Scribe interface
>
>
> Thanks,
>
> Denny Ye
>
>