Hi, Mike

Would you help check in this patch as Will agree with latest patch?
Thanks so much!

Best Regards
Leslin

2012/6/23 Will McQueen <[email protected]>

>    This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/5466/
>
> On June 22nd, 2012, 6:57 p.m., *Will McQueen* wrote:
>
> lgtm! Thanks Leslie.
>
>  Sorry, I meant 'Leslin' :-)
>
>
> - Will
>
> On June 22nd, 2012, 3:07 p.m., Leslin (Hong Xiang Lin) wrote:
>   Review request for Flume.
> By Leslin (Hong Xiang Lin).
>
> *Updated June 22, 2012, 3:07 p.m.*
> Description
>
> Change flume-ng script to find java from PATH first instead of using 
> bigtop-detect-javahome to get when JAVA_HOME does not set. It will helpful in 
> multi-java environment.
>
>   Testing
>
> test with normal start agent
>
>   *Bugs: * FLUME-1154 <https://issues.apache.org/jira/browse/FLUME-1154>
> Diffs
>
>    - bin/flume-ng (24a7d95)
>
> View Diff <https://reviews.apache.org/r/5466/diff/>
>



-- 



Best Regards

Leslin

Reply via email to