[ 
https://issues.apache.org/jira/browse/FLUME-1032?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13232888#comment-13232888
 ] 

[email protected] commented on FLUME-1032:
------------------------------------------------------


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

Ship it!


Looks good to me.

- Andrew


On 2012-03-15 21:42:49, Mike Percy wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/4368/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2012-03-15 21:42:49)
bq.  
bq.  
bq.  Review request for Flume.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  Fixing various build issues:
bq.  
bq.  1. Maven build against Hadoop 0.23 is misconfigured:
bq.    - Referenced Hadoop artifact (0.23-SNAPSHOT) does not exist in central 
repo (build fails due to this)
bq.    - slf4j-jcl gets pulled in, which breaks logging at runtime due to the 
inclusion of slf4j-log4j12 as well
bq.    - downstream dependencies which should be transitively included are 
explicitly referenced only for hadoop-0.23 (commons-logging and 
commons-configuration)
bq.    - hadoop-hdfs jar is missing
bq.    (I think much of this was broken-build stuff in the SNAPSHOT which was 
fixed for release)
bq.  
bq.  2. Several artifacts are missing from the -dist archive:
bq.    - SDK jar
bq.    - Legacy sources
bq.    - log4j appender client
bq.  
bq.  
bq.  This addresses bug FLUME-1032.
bq.      https://issues.apache.org/jira/browse/FLUME-1032
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    flume-ng-clients/flume-ng-log4jappender/pom.xml d1c443d 
bq.    flume-ng-dist/pom.xml f7e0dd9 
bq.    flume-ng-dist/src/main/assembly/dist.xml afe7a44 
bq.    flume-ng-dist/src/main/assembly/src.xml 21278dc 
bq.    flume-ng-sinks/flume-hdfs-sink/pom.xml 76d69a3 
bq.    pom.xml cf10727 
bq.  
bq.  Diff: https://reviews.apache.org/r/4368/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  Tested 3 profiles: default (hadoop-0.20.205), -Dhadoop.profile=100 
(hadoop-1.0.0), -Dhadoop.profile=23 (hadoop-0.23)
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Mike
bq.  
bq.


                
> Fix / clean up Flume NG build
> -----------------------------
>
>                 Key: FLUME-1032
>                 URL: https://issues.apache.org/jira/browse/FLUME-1032
>             Project: Flume
>          Issue Type: Improvement
>            Reporter: Mike Percy
>            Assignee: Mike Percy
>
> There are problems with the current Flume NG maven build setup:
> 1. Maven build against Hadoop 0.23 is misconfigured:
>    - Referenced Hadoop artifact does not exist in central repo (build fails 
> due to this)
>    - commons-logging gets pulled in, which causes slf4j errors at Flume 
> runtime due to multiple logger implementations (commons & log4j)
> 2. Several artifacts are missing from the -dist archive:
>    - SDK jar
>    - Legacy sources
>    - log4j appender client
> Patch forthcoming to address these issues.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to