Re: Review Request 52850: Icons don't display in Windows 10 Microsoft Edge or Internet Explorer 11

2016-11-03 Thread Vivek Ratnavel Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52850/ --- (Updated Nov. 4, 2016, 1:45 a.m.) Review request for Ambari, Jaimin Jetly, Zhe

Review Request 53454: AMBARI-18795: Ambari-server: Stack definition to specify which services support credential store

2016-11-03 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53454/ --- Review request for Ambari, Jonathan Hurley, Jayush Luniya, Nate Cole, Robert

Re: Review Request 53433: Add orchestration for HOST_ORDERED upgrades

2016-11-03 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53433/#review154789 --- Ship it! Ship It! - Jonathan Hurley On Nov. 3, 2016, 2:53

Re: Review Request 53433: Add orchestration for HOST_ORDERED upgrades

2016-11-03 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53433/ --- (Updated Nov. 3, 2016, 2:53 p.m.) Review request for Ambari, Alejandro

Re: Review Request 53433: Add orchestration for HOST_ORDERED upgrades

2016-11-03 Thread Nate Cole
> On Nov. 3, 2016, 1:09 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/HostOrderGrouping.java, > > lines 124-126 > > > > > > Could hosts be null if not

Re: Review Request 53436: Change the Wording of the HiveServer2 Non-Rolling Warning

2016-11-03 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53436/#review154780 --- Ship it! Ship It! - Nate Cole On Nov. 3, 2016, 12:53 p.m.,

Re: Review Request 53412: AMBARI-18778: Ambari-server: Blueprint enhancements to support enabling or disabling credential store

2016-11-03 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53412/#review154778 --- Ship it! Ship It! - Robert Nettleton On Nov. 3, 2016, 3:51

Re: Review Request 53412: AMBARI-18778: Ambari-server: Blueprint enhancements to support enabling or disabling credential store

2016-11-03 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53412/#review154769 --- Ship it! Ship It! - Jonathan Hurley On Nov. 3, 2016, 11:51

Re: Review Request 53433: Add orchestration for HOST_ORDERED upgrades

2016-11-03 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53433/#review154764 --- Ship it!

Re: Review Request 53436: Change the Wording of the HiveServer2 Non-Rolling Warning

2016-11-03 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53436/#review154762 --- Ship it! Ship It! - Alejandro Fernandez On Nov. 3, 2016,

Review Request 53436: Change the Wording of the HiveServer2 Non-Rolling Warning

2016-11-03 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53436/ --- Review request for Ambari and Nate Cole. Bugs: AMBARI-18789

Re: Review Request 53416: AMBARI-18746: Sqoop parameter hadoop_home is pointed to incorrect path in params_linux.py

2016-11-03 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53416/#review154759 --- Ship it! Ship It! - Alejandro Fernandez On Nov. 3, 2016,

Re: Review Request 53346: AMBARI-18765 - Services with invalid themes hang the UI when you attempt to add them

2016-11-03 Thread Tim Thorpe
> On Nov. 3, 2016, 4:21 p.m., Jayush Luniya wrote: > > Ship It! > > Jayush Luniya wrote: > Ok with the change, but does this need to be in 2.4.2 or fine with > keeping this in 2.5.0? Hi Jayush, Didn't see your reply until I had just finished pushing this into 2.4.2, 2.5 and trunk. -

Re: Review Request 53346: AMBARI-18765 - Services with invalid themes hang the UI when you attempt to add them

2016-11-03 Thread Jayush Luniya
> On Nov. 3, 2016, 4:21 p.m., Jayush Luniya wrote: > > Ship It! Ok with the change, but does this need to be in 2.4.2 or fine with keeping this in 2.5.0? - Jayush --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request 53346: AMBARI-18765 - Services with invalid themes hang the UI when you attempt to add them

2016-11-03 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53346/#review154748 --- Ship it! Ship It! - Jayush Luniya On Nov. 1, 2016, 6:47

Re: Review Request 53433: Add orchestration for HOST_ORDERED upgrades

2016-11-03 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53433/ --- (Updated Nov. 3, 2016, 12:18 p.m.) Review request for Ambari, Alejandro

Re: Review Request 53087: Log Search schema meta data loader fix

2016-11-03 Thread Miklos Gergely
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53087/ --- (Updated Nov. 3, 2016, 3:57 p.m.) Review request for Ambari, Oliver Szabo,

Re: Review Request 53412: AMBARI-18778: Ambari-server: Blueprint enhancements to support enabling or disabling credential store

2016-11-03 Thread Nahappan Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53412/ --- (Updated Nov. 3, 2016, 8:51 a.m.) Review request for Ambari, Jonathan Hurley,

Review Request 53433: Add orchestration for HOST_ORDERED upgrades

2016-11-03 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53433/ --- Review request for Ambari, Alejandro Fernandez and Jonathan Hurley. Bugs:

Re: Review Request 53430: HDP Upgrade fails when the cluster size is large

2016-11-03 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53430/#review154734 --- Ship it! Ship It! - Nate Cole On Nov. 3, 2016, 10:48 a.m.,

Re: Review Request 53430: HDP Upgrade fails when the cluster size is large

2016-11-03 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53430/#review154733 --- Ship it! Ship It! - Jonathan Hurley On Nov. 3, 2016, 10:48

Review Request 53430: HDP Upgrade fails when the cluster size is large

2016-11-03 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53430/ --- Review request for Ambari, Jonathan Hurley and Nate Cole. Bugs: AMBARI-18786

Re: Review Request 53334: AMBARI-18758. Livy configuration change for upgrade

2016-11-03 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53334/#review154728 --- Fix it, then Ship it! To update existng configurations during

Re: Review Request 53412: AMBARI-18778: Ambari-server: Blueprint enhancements to support enabling or disabling credential store

2016-11-03 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53412/#review154708 --- Fix it, then Ship it!

Re: Review Request 52839: Ability to enable the DEBUG/INFO logs in Ambari-server without server restart

2016-11-03 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52839/#review154706 --- What's the state of this review? Can it be closed? - Jonathan

Re: Review Request 53416: AMBARI-18746: Sqoop parameter hadoop_home is pointed to incorrect path in params_linux.py

2016-11-03 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53416/#review154705 --- Ship it! Ship It! - Tim Thorpe On Nov. 3, 2016, 1:09 a.m.,

Re: Review Request 53396: DB consistency check should inform user about warnings but not fail ambari-server start

2016-11-03 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53396/#review154696 --- Ship it! Ship It! - Dmytro Sen On Ноя. 2, 2016, 6:33 п.п.,

Re: Review Request 53373: Add validations and attributes to support HOST_ORDERED upgrades

2016-11-03 Thread Dmitro Lisnichenko
> On Nov. 2, 2016, 8:23 p.m., Dmitro Lisnichenko wrote: > > I see the upgrade pack for new upgrade type. Is downgrade possible? Is host > > order reversed during downgrade? > > Jonathan Hurley wrote: > Downgrades are possible, yes. Nothing is going to be "reversed" during a > HOST_ORDERED

Re: Review Request 53412: AMBARI-18778: Ambari-server: Blueprint enhancements to support enabling or disabling credential store

2016-11-03 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53412/#review154684 --- Ship it! Ship It! - Jayush Luniya On Nov. 2, 2016, 11:54

Re: Review Request 53283: AMBARI-18744: Ambari-server: REST API changes to GET and PUT credential store information

2016-11-03 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53283/#review154683 --- Ship it! Ship It! - Jayush Luniya On Nov. 2, 2016, 5:49

Re: Review Request 53216: AMBARI-18711: Ambari-server: DB changes to enable/disable credential store support

2016-11-03 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53216/#review154682 --- Ship it! Ship It! - Jayush Luniya On Oct. 31, 2016, 8:10