Re: Review Request 52260: AMBARI-18459: Print error messages if bulkcommand section of a component includes non-existing component

2016-09-27 Thread Tim Thorpe

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


Ship it!




Ship It!

- Tim Thorpe


On Sept. 26, 2016, 2:26 p.m., Di Li wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52260/
> ---
> 
> (Updated Sept. 26, 2016, 2:26 p.m.)
> 
> 
> Review request for Ambari and Tim Thorpe.
> 
> 
> Bugs: AMBARI-18459
> https://issues.apache.org/jira/browse/AMBARI-18459
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> This is to provide basic error message print outs when a user defines a 
> bulkcommand section in a service's metainfo.xml where he unknowingly addes a 
> component that doesn't exist. For example, adding a component called "master" 
> to HBASE_REGIONSERVER's bulkcommand section should see the server prints out 
> an error message.
> 
> 
> Diffs
> -
> 
>   
> ambari-server/src/main/java/org/apache/ambari/server/stack/ServiceModule.java 
> a77a22f 
>   ambari-server/src/main/java/org/apache/ambari/server/stack/StackModule.java 
> 93eeb7e 
> 
> Diff: https://reviews.apache.org/r/52260/diff/
> 
> 
> Testing
> ---
> 
> existing unit test.
> patched a trunk cluster with the change, 
> 1. modified HBase metainfo.xml at the common service level and see the error 
> messages showed up for all HDP stacks.
> 2. modified hbase metainfo.xml at both common service and HDP 2.5 level (2.5 
> level has the correct bulkcommand section), see the error messages for HDP 
> 2.0.6 up to HDP 2.4, no error in HDP 2.5 (as it has the correct bulkcommand 
> section)
> 
> 
> Thanks,
> 
> Di Li
> 
>



Review Request 52260: AMBARI-18459: Print error messages if bulkcommand section of a component includes non-existing component

2016-09-26 Thread Di Li

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

Review request for Ambari and Tim Thorpe.


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


Repository: ambari


Description
---

This is to provide basic error message print outs when a user defines a 
bulkcommand section in a service's metainfo.xml where he unknowingly addes a 
component that doesn't exist. For example, adding a component called "master" 
to HBASE_REGIONSERVER's bulkcommand section should see the server prints out an 
error message.


Diffs
-

  ambari-server/src/main/java/org/apache/ambari/server/stack/ServiceModule.java 
a77a22f 
  ambari-server/src/main/java/org/apache/ambari/server/stack/StackModule.java 
93eeb7e 

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


Testing
---

existing unit test.
patched a trunk cluster with the change, 
1. modified HBase metainfo.xml at the common service level and see the error 
messages showed up for all HDP stacks.
2. modified hbase metainfo.xml at both common service and HDP 2.5 level (2.5 
level has the correct bulkcommand section), see the error messages for HDP 
2.0.6 up to HDP 2.4, no error in HDP 2.5 (as it has the correct bulkcommand 
section)


Thanks,

Di Li