You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Andriy Babiichuk <ab...@hortonworks.com> on 2014/06/03 17:39:48 UTC

Review Request 22209: Usability: provide database connectivity checks in Customize Services page

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

Review request for Ambari and Andrii Tkach.


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


Repository: ambari


Description
-------

Integrate database connectivity checking in the Install/Add Services wizards' Customize Services page.


Diffs
-----

  ambari-web/app/assets/test/tests.js 8132f29 
  ambari-web/app/config.js 4709a6f 
  ambari-web/app/controllers/global/cluster_controller.js a1565e0 
  ambari-web/app/controllers/wizard/step7_controller.js e365fde 
  ambari-web/app/data/HDP2/global_properties.js 0922c1e 
  ambari-web/app/data/HDP2/site_properties.js d86d01d 
  ambari-web/app/messages.js 8b3fd94 
  ambari-web/app/models/cluster_states.js 320c3f4 
  ambari-web/app/models/service_config.js 8db9a19 
  ambari-web/app/styles/application.less 85dc6b0 
  ambari-web/app/templates/common/configs/service_config.hbs 15c930a 
  ambari-web/app/templates/common/error_log_body.hbs PRE-CREATION 
  ambari-web/app/templates/common/form/check_db_connection.hbs PRE-CREATION 
  ambari-web/app/utils/ajax/ajax.js 26b2de0 
  ambari-web/app/utils/db.js 24f95d0 
  ambari-web/app/utils/helper.js 5ec945a 
  ambari-web/app/views/wizard/controls_view.js 74eda07 
  ambari-web/test/utils/helper_test.js 3718aef 
  ambari-web/test/views/wizard/controls_view_test.js PRE-CREATION 

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


Testing
-------


Thanks,

Andriy Babiichuk


Re: Review Request 22209: Usability: provide database connectivity checks in Customize Services page

Posted by Andrii Tkach <at...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22209/#review44629
-----------------------------------------------------------

Ship it!


Ship It!

- Andrii Tkach


On June 3, 2014, 3:39 p.m., Andriy Babiichuk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/22209/
> -----------------------------------------------------------
> 
> (Updated June 3, 2014, 3:39 p.m.)
> 
> 
> Review request for Ambari and Andrii Tkach.
> 
> 
> Bugs: AMBARI-6009
>     https://issues.apache.org/jira/browse/AMBARI-6009
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Integrate database connectivity checking in the Install/Add Services wizards' Customize Services page.
> 
> 
> Diffs
> -----
> 
>   ambari-web/app/assets/test/tests.js 8132f29 
>   ambari-web/app/config.js 4709a6f 
>   ambari-web/app/controllers/global/cluster_controller.js a1565e0 
>   ambari-web/app/controllers/wizard/step7_controller.js e365fde 
>   ambari-web/app/data/HDP2/global_properties.js 0922c1e 
>   ambari-web/app/data/HDP2/site_properties.js d86d01d 
>   ambari-web/app/messages.js 8b3fd94 
>   ambari-web/app/models/cluster_states.js 320c3f4 
>   ambari-web/app/models/service_config.js 8db9a19 
>   ambari-web/app/styles/application.less 85dc6b0 
>   ambari-web/app/templates/common/configs/service_config.hbs 15c930a 
>   ambari-web/app/templates/common/error_log_body.hbs PRE-CREATION 
>   ambari-web/app/templates/common/form/check_db_connection.hbs PRE-CREATION 
>   ambari-web/app/utils/ajax/ajax.js 26b2de0 
>   ambari-web/app/utils/db.js 24f95d0 
>   ambari-web/app/utils/helper.js 5ec945a 
>   ambari-web/app/views/wizard/controls_view.js 74eda07 
>   ambari-web/test/utils/helper_test.js 3718aef 
>   ambari-web/test/views/wizard/controls_view_test.js PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/22209/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Andriy Babiichuk
> 
>