Re: Review Request 63962: Conditionally Rebuild MapReduce and Tez Tarballs with LZO if Enabled

2017-11-21 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63962/#review191645 --- Ship it! Ship It! - Nate Cole On Nov. 20, 2017, 2:27 p.m.,

Re: Review Request 63962: Conditionally Rebuild MapReduce and Tez Tarballs with LZO if Enabled

2017-11-21 Thread Jonathan Hurley
> On Nov. 20, 2017, 7:36 p.m., Dmytro Grinenko wrote: > > ambari-common/src/main/python/resource_management/libraries/functions/copy_tarball.py > > Lines 96 (patched) > > > > > > better to use function from the sudo

Re: Review Request 63962: Conditionally Rebuild MapReduce and Tez Tarballs with LZO if Enabled

2017-11-20 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63962/#review191550 --- Fix it, then Ship it!