Re: Review Request 64552: Fix misuses of os.path.dirname(path) in yarn.py

2017-12-20 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64552/#review194316 --- Ship it! Ship It! - Andrew Onischuk On Dec. 12, 2017, 7:36 p

Re: Review Request 64751: Fix Broken Symlinks on Stack Distribution

2017-12-20 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64751/#review194249 --- Fix it, then Ship it! ambari-server/src/main/resources/custom_

Re: Review Request 64722: [Patch Hive]webhcat: test_sqoop fails with hdfs:///hdp/apps/2.6.*/sqoop/sqoop.tar.gz does not exist

2017-12-20 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64722/#review194244 --- Ship it! Ship It! - Dmytro Grinenko On Dec. 19, 2017, 8:58 p

Re: Review Request 64722: [Patch Hive]webhcat: test_sqoop fails with hdfs:///hdp/apps/2.6.*/sqoop/sqoop.tar.gz does not exist

2017-12-20 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64722/#review194243 --- Ship it! Ship It! - Dmitro Lisnichenko On Dec. 19, 2017, 10:

Review Request 64751: Fix Broken Symlinks on Stack Distribution

2017-12-20 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64751/ --- Review request for Ambari, Jonathan Hurley and Nate Cole. Bugs: AMBARI-22678

Re: Review Request 64747: Don't call pprint.pformat unnecessarily in Ambari agent

2017-12-20 Thread Attila Magyar
> On Dec. 20, 2017, 12:44 p.m., Andrew Onischuk wrote: > > ambari-agent/src/main/python/ambari_agent/ActionQueue.py > > Line 214 (original), 215 (patched) > > > > > > Not a issue. Just wondering if that does make a c

Re: Review Request 64747: Don't call pprint.pformat unnecessarily in Ambari agent

2017-12-20 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64747/#review194235 --- Ship it! Ship It! - Andrew Onischuk On Dec. 20, 2017, 12:18

Re: Review Request 64747: Don't call pprint.pformat unnecessarily in Ambari agent

2017-12-20 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64747/#review194234 --- ambari-agent/src/main/python/ambari_agent/ActionQueue.py Line 214

Re: Review Request 64747: Don't call pprint.pformat unnecessarily in Ambari agent

2017-12-20 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64747/#review194232 --- Ship it! Ship It! - Attila Doroszlai On Dec. 20, 2017, 1:18

Review Request 64747: Don't call pprint.pformat unnecessarily in Ambari agent

2017-12-20 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64747/ --- Review request for Ambari, Attila Doroszlai, Andrew Onischuk, and Eugene Chekans

Re: Review Request 64707: AMBARI-22669 Ranger stack script changes to fix missing directory failure for blueprint installation

2017-12-20 Thread Vishal Suvagia via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64707/#review194229 --- Ship it! - Vishal Suvagia On Dec. 19, 2017, 10:19 a.m., Mugdh