Re: Review Request 60986: AMBARI-21528. Zookeeper server has incorrect memory setting, missing m in Xmx value

2017-07-19 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60986/#review181014 --- Ship it! Ship It! - Sid Wagle On July 20, 2017, 12:13 a.m.,

Re: Review Request 60992: Service Checks During Upgrades Should Use Desired Stack

2017-07-19 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60992/#review181013 --- Ship it! Ship It! - Sid Wagle On July 20, 2017, 2:06 a.m.,

Review Request 60992: Service Checks During Upgrades Should Use Desired Stack

2017-07-19 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60992/ --- Review request for Ambari, Alejandro Fernandez and Sid Wagle. Bugs:

Re: Review Request 60986: AMBARI-21528. Zookeeper server has incorrect memory setting, missing m in Xmx value

2017-07-19 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60986/#review181003 --- Ship it! Ship It! - Jonathan Hurley On July 19, 2017, 8:13

Re: Review Request 60986: AMBARI-21528. Zookeeper server has incorrect memory setting, missing m in Xmx value

2017-07-19 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60986/ --- (Updated July 20, 2017, 12:13 a.m.) Review request for Ambari, Di Li, Jonathan

Re: Review Request 60986: AMBARI-21528. Zookeeper server has incorrect memory setting, missing m in Xmx value

2017-07-19 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60986/#review180997 ---

Re: Review Request 60986: AMBARI-21528. Zookeeper server has incorrect memory setting, missing m in Xmx value

2017-07-19 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60986/#review180991 ---

Re: Review Request 60982: ZKFC Doesn't Update its Version On Some Upgrades

2017-07-19 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60982/#review180984 --- Ship it! Ship It! - Alejandro Fernandez On July 19, 2017,

Re: Review Request 60982: ZKFC Doesn't Update its Version On Some Upgrades

2017-07-19 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60982/#review180979 --- Ship it! Ship It! - Sid Wagle On July 19, 2017, 8:50 p.m.,

Review Request 60982: ZKFC Doesn't Update its Version On Some Upgrades

2017-07-19 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60982/ --- Review request for Ambari, Alejandro Fernandez and Sid Wagle. Bugs:

Review Request 60981: EU failure: Oozie service check fails if Falcon is not installed

2017-07-19 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60981/ --- Review request for Ambari, Andrew Onischuk, Dmytro Sen, and Sid Wagle. Bugs:

Re: Review Request 60975: Installation Commands On New Clusters Don't Send Down Correct Versions

2017-07-19 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60975/#review180957 --- Ship it! Ship It! - Robert Levas On July 19, 2017, 2:34

Re: Review Request 60975: Installation Commands On New Clusters Don't Send Down Correct Versions

2017-07-19 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60975/ --- (Updated July 19, 2017, 2:34 p.m.) Review request for Ambari, Alejandro

Review Request 60975: Installation Commands On New Clusters Don't Send Down Correct Versions

2017-07-19 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60975/ --- Review request for Ambari, Alejandro Fernandez and Robert Levas. Bugs:

Re: Review Request 60972: Include/exclude files update is triggered when master is not started if cluster is deployed via BP

2017-07-19 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60972/#review180924 --- Ship it! Ship It! - Vitalyi Brodetskyi On Липень 19, 2017,

Review Request 60972: Include/exclude files update is triggered when master is not started if cluster is deployed via BP

2017-07-19 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60972/ --- Review request for Ambari, Aravindan Vijayan, Sumit Mohanty, Sid Wagle, and

Re: Review Request 60966: Log Search docker test environment build front/backend only

2017-07-19 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60966/#review180923 --- Ship it! Ship It! - Robert Nettleton On July 19, 2017,

Re: Review Request 60968: Fix custom actions to handle changed config paths

2017-07-19 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60968/#review180921 --- Ship it! Ship It! - Vitalyi Brodetskyi On Липень 19, 2017,

Re: Review Request 60969: Ambari server logs NPE with no additional stack trace on any host component start/stop command

2017-07-19 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60969/#review180920 --- Ship it! Ship It! - Andrew Onischuk On July 19, 2017, 1:35

Review Request 60969: Ambari server logs NPE with no additional stack trace on any host component start/stop command

2017-07-19 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60969/ --- Review request for Ambari, Andrew Onischuk and Sumit Mohanty. Bugs:

Review Request 60968: Fix custom actions to handle changed config paths

2017-07-19 Thread Andrew Onischuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60968/ --- Review request for Ambari, Myroslav Papirkovskyy, Sid Wagle, and Vitalyi

Re: Review Request 60966: Log Search docker test environment build front/backend only

2017-07-19 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60966/ --- (Updated July 19, 2017, 12:22 p.m.) Review request for Ambari, Oliver Szabo

Re: Review Request 60966: Log Search docker test environment build front/backend only

2017-07-19 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60966/#review180918 --- Ship it! Ship It! (note: add some docs for the flags) -

Review Request 60966: Log Search docker test environment build front/backend only

2017-07-19 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60966/ --- Review request for Ambari, Oliver Szabo and Robert Nettleton. Bugs: