You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@ambari.apache.org by alice alice <zh...@cmss.chinamobile.com> on 2016/12/06 02:31:39 UTC

Re: Review Request 51640: After staring rm ha, the config of 'yarn.resoucemanager.admin.address.rm1' and 'yarn.resoucemanager.admin.address.rm2' are missing.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51640/
-----------------------------------------------------------

(Updated \u5341\u4e8c\u6708 6, 2016, 2:31 a.m.)


Review request for Ambari, Jonathan Hurley and Vitalyi Brodetskyi.


Changes
-------

when deploying rm ha,the config of 'yarn.resoucemanager.admin.address.rm1' and 'yarn.resoucemanager.admin.address.rm2' are missing;Maybe we need to config these when starting rm ha.


Summary (updated)
-----------------

After staring rm ha,the config of 'yarn.resoucemanager.admin.address.rm1' and 'yarn.resoucemanager.admin.address.rm2' are missing.


Bugs: AMBARI-18312
    https://issues.apache.org/jira/browse/AMBARI-18312


Repository: ambari


Description
-------

1?when deploying rm ha,the config of 'yarn.resoucemanager.admin.address.rm1' and 'yarn.resoucemanager.admin.address.rm2' are missing; so we need to config these.


Diffs
-----

  ambari-web/app/controllers/main/admin/highAvailability/resourceManager/step3_controller.js 36dcfb8 
  ambari-web/app/data/HDP2/rm_ha_properties.js fb1bd13 
  ambari-web/app/utils/configs/rm_ha_config_initializer.js dac68d9 

Diff: https://reviews.apache.org/r/51640/diff/


Testing
-------

done it


Thanks,

alice alice


Re: Review Request 51640: After staring rm ha, the config of 'yarn.resoucemanager.admin.address.rm1' and 'yarn.resoucemanager.admin.address.rm2' are missing.

Posted by alice alice <zh...@cmss.chinamobile.com>.

> On \u5341\u4e8c\u6708 6, 2016, 3:03 p.m., Jonathan Hurley wrote:
> > I normally don't review web client requests since I have little experience with that part of the codebase...

OK,thank you all the same for your swift respondence.


- alice


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51640/#review158159
-----------------------------------------------------------


On \u5341\u4e8c\u6708 6, 2016, 2:31 a.m., alice alice wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51640/
> -----------------------------------------------------------
> 
> (Updated \u5341\u4e8c\u6708 6, 2016, 2:31 a.m.)
> 
> 
> Review request for Ambari, Jonathan Hurley and Vitalyi Brodetskyi.
> 
> 
> Bugs: AMBARI-18312
>     https://issues.apache.org/jira/browse/AMBARI-18312
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> 1?when deploying rm ha,the config of 'yarn.resoucemanager.admin.address.rm1' and 'yarn.resoucemanager.admin.address.rm2' are missing; so we need to config these.
> 
> 
> Diffs
> -----
> 
>   ambari-web/app/controllers/main/admin/highAvailability/resourceManager/step3_controller.js 36dcfb8 
>   ambari-web/app/data/HDP2/rm_ha_properties.js fb1bd13 
>   ambari-web/app/utils/configs/rm_ha_config_initializer.js dac68d9 
> 
> Diff: https://reviews.apache.org/r/51640/diff/
> 
> 
> Testing
> -------
> 
> done it
> 
> 
> Thanks,
> 
> alice alice
> 
>


Re: Review Request 51640: After staring rm ha, the config of 'yarn.resoucemanager.admin.address.rm1' and 'yarn.resoucemanager.admin.address.rm2' are missing.

Posted by Jonathan Hurley <jh...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51640/#review158159
-----------------------------------------------------------



I normally don't review web client requests since I have little experience with that part of the codebase...

- Jonathan Hurley


On Dec. 5, 2016, 9:31 p.m., alice alice wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51640/
> -----------------------------------------------------------
> 
> (Updated Dec. 5, 2016, 9:31 p.m.)
> 
> 
> Review request for Ambari, Jonathan Hurley and Vitalyi Brodetskyi.
> 
> 
> Bugs: AMBARI-18312
>     https://issues.apache.org/jira/browse/AMBARI-18312
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> 1?when deploying rm ha,the config of 'yarn.resoucemanager.admin.address.rm1' and 'yarn.resoucemanager.admin.address.rm2' are missing; so we need to config these.
> 
> 
> Diffs
> -----
> 
>   ambari-web/app/controllers/main/admin/highAvailability/resourceManager/step3_controller.js 36dcfb8 
>   ambari-web/app/data/HDP2/rm_ha_properties.js fb1bd13 
>   ambari-web/app/utils/configs/rm_ha_config_initializer.js dac68d9 
> 
> Diff: https://reviews.apache.org/r/51640/diff/
> 
> 
> Testing
> -------
> 
> done it
> 
> 
> Thanks,
> 
> alice alice
> 
>


Re: Review Request 51640: After staring rm ha, the config of 'yarn.resoucemanager.admin.address.rm1' and 'yarn.resoucemanager.admin.address.rm2' are missing.

Posted by alice alice <zh...@cmss.chinamobile.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51640/
-----------------------------------------------------------

(Updated \u5341\u4e8c\u6708 7, 2016, 12:53 a.m.)


Review request for Ambari, Zhe (Joe) Wang and Yusaku Sako.


Bugs: AMBARI-18312
    https://issues.apache.org/jira/browse/AMBARI-18312


Repository: ambari


Description (updated)
-------

1. when deploying rm ha,the config of 'yarn.resoucemanager.admin.address.rm1' and 'yarn.resoucemanager.admin.address.rm2' are missing; so we need to config these.


Diffs
-----

  ambari-web/app/controllers/main/admin/highAvailability/resourceManager/step3_controller.js 36dcfb8 
  ambari-web/app/data/HDP2/rm_ha_properties.js fb1bd13 
  ambari-web/app/utils/configs/rm_ha_config_initializer.js dac68d9 

Diff: https://reviews.apache.org/r/51640/diff/


Testing
-------

done it


Thanks,

alice alice