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


Ship it!




Ship It!

- Alexandr Antonenko


On April 13, 2016, 1:54 p.m., Di Li wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46144/
> -----------------------------------------------------------
> 
> (Updated April 13, 2016, 1:54 p.m.)
> 
> 
> Review request for Ambari and Alexandr Antonenko.
> 
> 
> Bugs: AMBARI-15844
>     https://issues.apache.org/jira/browse/AMBARI-15844
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Same named properties are currently disallowed for the same configType for a 
> given service, this jira proposes to display a warning to remind user the 
> existence of same-named properties for the service among all configTypes.
> 
> 
> Diffs
> -----
> 
>   ambari-web/app/messages.js 2e4f648 
>   ambari-web/app/templates/common/configs/addPropertyWindow.hbs 61572fd 
>   ambari-web/app/views/common/configs/service_configs_by_category_view.js 
> b5d5b45 
> 
> Diff: https://reviews.apache.org/r/46144/diff/
> 
> 
> Testing
> -------
> 
> patch a truck cluster with UI changes, 
> 
> 1. add "zookeeper.znode.parent" to hbase-site, notice the UI shows the error 
> message about the property is a dup.
> This is to test the current error check still works.
> 
> 2. add "zookeeper.znode.parent" to hbase-policy.xml, notice the UI shows a 
> warning this time, about a same-named property is defined in hbase-site.xml. 
> This is to test the new behavior introduced via this JIRA.
> 
> 
> Thanks,
> 
> Di Li
> 
>

Reply via email to