Hi Leslin! Done. 

Best,
Mike


On Sunday, June 24, 2012 at 3:42 AM, Leslin wrote:

> 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] (mailto:[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