Re: Review Request 64948: Improved service/component dependency support (component level)

2018-01-04 Thread Attila Magyar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64948/ --- (Updated Jan. 5, 2018, 7:54 a.m.) Review request for Ambari, Aleksandr

Re: Review Request 64951: Idempotent issue on Ambari Upgrade, renameServiceDeletedColumn failed with column already exists exception

2018-01-04 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64951/#review194838 ---

Re: Review Request 64956: Update Hadoop RPC Encryption Properties During Kerberization and Upgrade

2018-01-04 Thread Dmytro Grinenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64956/#review194837 --- Ship it! Ship It! - Dmytro Grinenko On Jan. 4, 2018, 8:16

Re: Review Request 64956: Update Hadoop RPC Encryption Properties During Kerberization and Upgrade

2018-01-04 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64956/#review194787 --- Ship it! Ship It! - Nate Cole On Jan. 4, 2018, 3:16 p.m.,

Review Request 64956: Update Hadoop RPC Encryption Properties During Kerberization and Upgrade

2018-01-04 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64956/ --- Review request for Ambari, Dmytro Grinenko, Dmitro Lisnichenko, and Nate Cole.

Re: Review Request 64769: AMBARI-22406 : UI for verifying if repo exist for new OS family and user prompt for entering HDP and HDP-UTILs repo URLs

2018-01-04 Thread Nate Cole
> On Dec. 22, 2017, 9:51 a.m., Nate Cole wrote: > > ambari-web/app/controllers/wizard/step3_controller.js > > Lines 863-880 (patched) > > > > > > I'm no UI expert, but this entire new block seems to be doing repo >

Re: Review Request 64952: Disabling Kerberos after enabled during Blueprint install fails with missing data directory error

2018-01-04 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64952/#review194769 --- Ship it! Ship It! - Robert Levas On Jan. 4, 2018, 12:06

Review Request 64952: Disabling Kerberos after enabled during Blueprint install fails with missing data directory error

2018-01-04 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64952/ --- Review request for Ambari and Robert Levas. Bugs: AMBARI-22629

Review Request 64951: Idempotent issue on Ambari Upgrade, renameServiceDeletedColumn failed with column already exists exception

2018-01-04 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64951/ --- Review request for Ambari, Andrew Onischuk, Dmytro Grinenko, and Dmytro Sen.

Re: Review Request 64769: AMBARI-22406 : UI for verifying if repo exist for new OS family and user prompt for entering HDP and HDP-UTILs repo URLs

2018-01-04 Thread Yussuf Shaikh
> On Dec. 22, 2017, 2:51 p.m., Nate Cole wrote: > > ambari-web/app/controllers/wizard/step3_controller.js > > Lines 863-880 (patched) > > > > > > I'm no UI expert, but this entire new block seems to be doing repo >

Re: Review Request 64950: Ambari Upgrade failed in case if database have no cluster_version table already. The issue with idempotence

2018-01-04 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64950/#review194766 --- Ship it! Ship It! - Jonathan Hurley On Jan. 4, 2018, 9:44

Re: Review Request 64769: AMBARI-22406 : UI for verifying if repo exist for new OS family and user prompt for entering HDP and HDP-UTILs repo URLs

2018-01-04 Thread Nate Cole
> On Dec. 22, 2017, 9:51 a.m., Nate Cole wrote: > > ambari-web/app/controllers/wizard/step3_controller.js > > Lines 863-880 (patched) > > > > > > I'm no UI expert, but this entire new block seems to be doing repo >

Re: Review Request 64887: AMBARI-22403 Read the JAVA_HOME depending on the OS family during Service install (action & command) and upgrade

2018-01-04 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64887/#review194764 --- This has been pushed. Please close this review and jira. - Nate

Re: Review Request 64887: AMBARI-22403 Read the JAVA_HOME depending on the OS family during Service install (action & command) and upgrade

2018-01-04 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64887/#review194763 --- Ship it! Ship It! - Nate Cole On Jan. 3, 2018, 2:03 a.m.,

Re: Review Request 64887: AMBARI-22403 Read the JAVA_HOME depending on the OS family during Service install (action & command) and upgrade

2018-01-04 Thread Nate Cole
> On Jan. 2, 2018, 11:51 a.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/utils/StageUtils.java > > Line 633 (original) > > > > > > It's much more convenient to keep JAVA_HOME in one

Re: Review Request 64948: Improved service/component dependency support (component level)

2018-01-04 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64948/#review194757 --- Fix it, then Ship it! Ship It!

Re: Review Request 64950: Ambari Upgrade failed in case if database have no cluster_version table already. The issue with idempotence

2018-01-04 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64950/#review194759 --- Ship it! Ship It! - Nate Cole On Jan. 4, 2018, 9:44 a.m.,

Re: Review Request 64950: Ambari Upgrade failed in case if database have no cluster_version table already. The issue with idempotence

2018-01-04 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64950/ --- (Updated Jan. 4, 2018, 4:44 p.m.) Review request for Ambari, Jonathan Hurley

Review Request 64950: Ambari Upgrade failed in case if database have no cluster_version table already. The issue with idempotence

2018-01-04 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64950/ --- Review request for Ambari, Jonathan Hurley and Nate Cole. Bugs: AMBARI-22723

Re: Review Request 64769: AMBARI-22406 : UI for verifying if repo exist for new OS family and user prompt for entering HDP and HDP-UTILs repo URLs

2018-01-04 Thread Sonia Garudi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64769/ --- (Updated Jan. 4, 2018, 12:07 p.m.) Review request for Ambari and Nate Cole.