You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@flume.apache.org by Mike Percy <mp...@gmail.com> on 2012/02/08 02:25:54 UTC
Review Request: FLUME-955 Add Apache License header to AvroLegacySource.java
to fix mvn install Rat error
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3785/
-----------------------------------------------------------
Review request for Flume.
Summary
-------
Simply add Apache License header to AvroLegacySource.java
This addresses bug FLUME-955.
https://issues.apache.org/jira/browse/FLUME-955
Diffs
-----
flume-ng-legacy-sources/flume-avro-source/src/main/java/org/apache/flume/source/avroLegacy/AvroLegacySource.java e7308c9
Diff: https://reviews.apache.org/r/3785/diff
Testing
-------
Checked out pristine copy of flume-728 branch, applied the patch, and mvn clean & install went through smoothly.
Thanks,
Mike
Re: Review Request: FLUME-955 Add Apache License header to
AvroLegacySource.java to fix mvn install Rat error
Posted by Arvind Prabhakar <ar...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3785/#review4891
-----------------------------------------------------------
Ship it!
+1. Please attach the patch to the Jira.
- Arvind
On 2012-02-08 01:25:54, Mike Percy wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/3785/
> -----------------------------------------------------------
>
> (Updated 2012-02-08 01:25:54)
>
>
> Review request for Flume.
>
>
> Summary
> -------
>
> Simply add Apache License header to AvroLegacySource.java
>
>
> This addresses bug FLUME-955.
> https://issues.apache.org/jira/browse/FLUME-955
>
>
> Diffs
> -----
>
> flume-ng-legacy-sources/flume-avro-source/src/main/java/org/apache/flume/source/avroLegacy/AvroLegacySource.java e7308c9
>
> Diff: https://reviews.apache.org/r/3785/diff
>
>
> Testing
> -------
>
> Checked out pristine copy of flume-728 branch, applied the patch, and mvn clean & install went through smoothly.
>
>
> Thanks,
>
> Mike
>
>