Re: Review Request 46851: Express Ugrade stuck on getting status on 1600 node cluster

2016-04-29 Thread Nate Cole
> On April 29, 2016, 8:17 p.m., Sid Wagle wrote: > > We did add a feature recently to allow blueprint based deploy with 0 hosts > > added to the cluster and thereby it might be possible to have no requests > > created. Not sure if it is relevant to this change though. I tested that scenario

Re: Review Request 46857: 'Restart All' for HAWQ should follow role_command_order

2016-04-29 Thread Matt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46857/#review131234 --- Ship it! Ship It! - Matt On April 29, 2016, 3:20 p.m., Lav

Review Request 46861: AMBARI-16193 Message with "null" shows up in showRestartRequiredPopup

2016-04-29 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46861/ --- Review request for Ambari, Alexandr Antonenko, Jaimin Jetly, Oleg Nechiporenko,

Re: Review Request 46860: Move RM wizard should update HAWQ related parameters

2016-04-29 Thread bhuvnesh chaudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46860/#review131232 ---

Re: Review Request 46860: Move RM wizard should update HAWQ related parameters

2016-04-29 Thread bhuvnesh chaudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46860/#review131230 ---

Re: Review Request 46851: Express Ugrade stuck on getting status on 1600 node cluster

2016-04-29 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46851/#review131229 --- Ship it! We did add a feature recently to allow blueprint

Re: Review Request 46852: HAWQ Configuration should be updated whenever Namenode is being moved using the move namenode wizard

2016-04-29 Thread jun aoki
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46852/#review131228 --- Ship it! Ship It! - jun aoki On April 29, 2016, 8:50 p.m.,

Re: Review Request 46852: HAWQ Configuration should be updated whenever Namenode is being moved using the move namenode wizard

2016-04-29 Thread bhuvnesh chaudhary
> On April 29, 2016, 9:12 p.m., jun aoki wrote: > > ambari-web/app/utils/configs/move_namenode_config_initializer.js, line 113 > > > > > > Did you test if this works when HAWQ is not installed? > > bhuvnesh

Review Request 46860: Move RM wizard should update HAWQ related parameters

2016-04-29 Thread jun aoki
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46860/ --- Review request for Ambari. Bugs: AMBARI-16174

Re: Review Request 46812: Topology host info is not cleared when a host is removed

2016-04-29 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46812/#review131224 --- Ship it! Ship It! - Alejandro Fernandez On April 29, 2016,

Re: Review Request 46460: AMBARI-16002 Integrate Red Hat Satellite option in Ambari Web

2016-04-29 Thread Zhe (Joe) Wang
> On April 29, 2016, 11:18 p.m., Jaimin Jetly wrote: > > ambari-web/app/mappers/stack_mapper.js, line 140 > > > > > > A minor thing. This line can be changed to > > stack.use_redhat_satellite = > >

Re: Review Request 46460: AMBARI-16002 Integrate Red Hat Satellite option in Ambari Web

2016-04-29 Thread Zhe (Joe) Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46460/ --- (Updated April 29, 2016, 11:12 p.m.) Review request for Ambari, Jaimin Jetly,

Re: Review Request 46460: AMBARI-16002 Integrate Red Hat Satellite option in Ambari Web

2016-04-29 Thread Jaimin Jetly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46460/#review131215 --- Ship it! ambari-web/app/mappers/stack_mapper.js (line 140)

Re: Review Request 46851: Express Ugrade stuck on getting status on 1600 node cluster

2016-04-29 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46851/ --- (Updated April 29, 2016, 7:08 p.m.) Review request for Ambari, Jonathan Hurley

Re: Review Request 46853: clean up import * : Accumulo, Ambari-metrics, Atlas, Falcon, Flume and Ganglia

2016-04-29 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46853/#review131208 ---

Re: Review Request 46853: clean up import * : Accumulo, Ambari-metrics, Atlas, Falcon, Flume and Ganglia

2016-04-29 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46853/#review131207 ---

Review Request 46857: 'Restart All' for HAWQ should follow role_command_order

2016-04-29 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46857/ --- Review request for Ambari, Alexander Denissov, bhuvnesh chaudhary, Goutam Tadi,

Re: Review Request 46727: [AMBARI-15612] Add Livy to HDP 2.5 as slave component of Spark

2016-04-29 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46727/#review131201 ---

Re: Review Request 46727: [AMBARI-15612] Add Livy to HDP 2.5 as slave component of Spark

2016-04-29 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46727/#review131200 ---

Re: Review Request 46727: [AMBARI-15612] Add Livy to HDP 2.5 as slave component of Spark

2016-04-29 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46727/#review131199 ---

Re: Review Request 46852: HAWQ Configuration should be updated whenever Namenode is being moved using the move namenode wizard

2016-04-29 Thread bhuvnesh chaudhary
> On April 29, 2016, 9:12 p.m., jun aoki wrote: > > ambari-web/app/utils/configs/move_namenode_config_initializer.js, line 113 > > > > > > Did you test if this works when HAWQ is not installed? Did not test it yet

Re: Review Request 46852: HAWQ Configuration should be updated whenever Namenode is being moved using the move namenode wizard

2016-04-29 Thread bhuvnesh chaudhary
> On April 29, 2016, 9:16 p.m., jun aoki wrote: > > ambari-web/app/controllers/main/service/reassign/step4_controller.js, line > > 83 > > > > > > Do you know how related services (or HAWQ) are restared after > >

Re: Review Request 46852: HAWQ Configuration should be updated whenever Namenode is being moved using the move namenode wizard

2016-04-29 Thread jun aoki
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46852/#review131191 ---

Re: Review Request 46852: HAWQ Configuration should be updated whenever Namenode is being moved using the move namenode wizard

2016-04-29 Thread jun aoki
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46852/#review131189 ---

Re: Review Request 46800: HAWQ Configuration should be updated whenever Namenode is being moved.

2016-04-29 Thread jun aoki
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46800/#review131186 --- Ship it! Ship It! - jun aoki On April 29, 2016, 8:46 p.m.,

Re: Review Request 44210: AMBARI-15226 - The stack advisor should be pushed down to the services

2016-04-29 Thread Alexander Denissov
> On April 29, 2016, 12:12 a.m., Alexander Denissov wrote: > > Do we have a branch cut with these changes in so that we can test HAWQ and > > PXF with this new logic ? After testing the branch can be merged to trunk > > and we will avoid any major surprises. > > Tim Thorpe wrote: > There

Re: Review Request 46800: HAWQ Configuration should be updated whenever Namenode is being moved.

2016-04-29 Thread Alexander Denissov
> On April 29, 2016, 8:59 p.m., Alexander Denissov wrote: > > Ship It! Do we also need to move PXF to the new namenode host ? - Alexander --- This is an automatically generated e-mail. To reply, visit:

Review Request 46852: HAWQ Configuration should be updated whenever Namenode is being moved using the move namenode wizard

2016-04-29 Thread bhuvnesh chaudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46852/ --- Review request for Ambari, Alexander Denissov, jun aoki, Matt, and Oleksandr

Re: Review Request 46800: HAWQ Configuration should be updated whenever Namenode is being moved.

2016-04-29 Thread bhuvnesh chaudhary
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46800/ --- (Updated April 29, 2016, 8:46 p.m.) Review request for Ambari, Alexander

Re: Review Request 46742: Use additional flags for HAWQ Check

2016-04-29 Thread Lav Jain
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46742/#review131178 --- Ship it! Ship It! - Lav Jain On April 29, 2016, 6:48 p.m.,

Re: Review Request 46851: Express Ugrade stuck on getting status on 1600 node cluster

2016-04-29 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46851/#review131176 --- Ship it! Ship It! - Alejandro Fernandez On April 29, 2016,

Re: Review Request 46851: Express Ugrade stuck on getting status on 1600 node cluster

2016-04-29 Thread Nate Cole
> On April 29, 2016, 4:05 p.m., Jonathan Hurley wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StageResourceProvider.java, > > lines 223-237 > > > > > > Something to check is

Re: Review Request 44210: AMBARI-15226 - The stack advisor should be pushed down to the services

2016-04-29 Thread Tim Thorpe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44210/ --- (Updated April 29, 2016, 8:11 p.m.) Review request for Ambari, Alexander

Re: Review Request 46851: Express Ugrade stuck on getting status on 1600 node cluster

2016-04-29 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46851/#review131173 --- Ship it!

Review Request 46851: Express Ugrade stuck on getting status on 1600 node cluster

2016-04-29 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46851/ --- Review request for Ambari, Jonathan Hurley and Sid Wagle. Bugs: AMBARI-16185

Re: Review Request 44210: AMBARI-15226 - The stack advisor should be pushed down to the services

2016-04-29 Thread Tim Thorpe
> On April 29, 2016, 12:11 a.m., Alexander Denissov wrote: > > ambari-server/src/main/java/org/apache/ambari/server/stack/CommonServiceDirectory.java, > > line 56 > > > > > > should we not have a convention at all

Re: Review Request 46742: Use additional flags for HAWQ Check

2016-04-29 Thread Goutam Tadi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46742/#review131161 --- Ship it! Ship It! - Goutam Tadi On April 29, 2016, 6:48

Re: Review Request 46833: AMBARI-16182 Add new component level identity for RANGER_KMS in kerberos.json for stack 2.5

2016-04-29 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46833/#review131150 ---

Re: Review Request 46833: AMBARI-16182 Add new component level identity for RANGER_KMS in kerberos.json for stack 2.5

2016-04-29 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46833/#review131149 ---

Re: Review Request 46760: AMBARI-16149. Support for LLAP alert in Ambari

2016-04-29 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46760/#review131142 --- Fix it, then Ship it!

Re: Review Request 44210: AMBARI-15226 - The stack advisor should be pushed down to the services

2016-04-29 Thread Alexander Denissov
> On April 28, 2016, 10:54 p.m., Alexander Denissov wrote: > > ambari-server/src/main/resources/common-services/HAWQ/2.0.0/service_advisor.py, > > line 110 > > > > > > if we make isLocalHost utility function

Re: Review Request 44210: AMBARI-15226 - The stack advisor should be pushed down to the services

2016-04-29 Thread Tim Thorpe
> On April 28, 2016, 10:54 p.m., Alexander Denissov wrote: > > ambari-server/src/main/resources/common-services/HAWQ/2.0.0/service_advisor.py, > > line 110 > > > > > > if we make isLocalHost utility function

Re: Review Request 46692: Improvements to DB consistency check

2016-04-29 Thread Dmytro Sen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46692/#review131107 --- Ship it! Ship It! - Dmytro Sen On Апрель 29, 2016, 3:25

Re: Review Request 46692: Improvements to DB consistency check

2016-04-29 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46692/ --- (Updated Квітень 29, 2016, 3:25 після полудня) Review request for Ambari,

Re: Review Request 46833: AMBARI-16182 Add new component level identity for RANGER_KMS in kerberos.json for stack 2.5

2016-04-29 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46833/#review131102 --- Ship it! Ship It! - Robert Levas On April 29, 2016, 10:06

Re: Review Request 44210: AMBARI-15226 - The stack advisor should be pushed down to the services

2016-04-29 Thread Tim Thorpe
> On April 29, 2016, 12:11 a.m., Alexander Denissov wrote: > > ambari-server/src/main/java/org/apache/ambari/server/stack/CommonServiceDirectory.java, > > line 56 > > > > > > should we not have a convention at all

Review Request 46833: AMBARI-16182 Add new component level identity for RANGER_KMS in kerberos.json for stack 2.5

2016-04-29 Thread Mugdha Varadkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46833/ --- Review request for Ambari, Alejandro Fernandez, Gautam Borad, Jonathan Hurley,

Re: Review Request 46797: AMBARI-16082. Test Refresh Phase 1 (#3) (jonathanhurley)

2016-04-29 Thread Jonathan Hurley
> On April 29, 2016, 8:10 a.m., Nate Cole wrote: > > Thanks for helping me find sh!t that wasn't happening in my env. Sure thing! I almost fell out of my chair when I saw that static injector living inside of the ECW. - Jonathan ---

Re: Review Request 46727: [AMBARI-15612] Add Livy to HDP 2.5 as slave component of Spark

2016-04-29 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46727/#review131041 ---

Re: Review Request 46820: 'llap' queue minimum recommended size is not persistent across HSI on and off.

2016-04-29 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46820/#review131083 --- Ship it! Ship It! - Sumit Mohanty On April 29, 2016, 11:08