> On April 22, 2014, 2:41 p.m., Sumit Mohanty wrote:
> > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java,
> >  line 560
> > <https://reviews.apache.org/r/20263/diff/4/?file=562290#file562290line560>
> >
> >     Is this for gmonds?

yes


- Dmytro


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


On April 22, 2014, 2:49 p.m., Dmytro Sen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20263/
> -----------------------------------------------------------
> 
> (Updated April 22, 2014, 2:49 p.m.)
> 
> 
> Review request for Ambari, Dmitro Lisnichenko, Dmytro Shkvyra, and Sumit 
> Mohanty.
> 
> 
> Bugs: AMBARI-5445
>     https://issues.apache.org/jira/browse/AMBARI-5445
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Any services with lines below in metainfo.xml will have stale_config == true 
> if any new host components will be added or deleted.
> <metainfo>
> <service>
> <monitoringService>true</monitoringService>
> </service>
> </metainfo>
> 
> 
> Diffs
> -----
> 
>   
> ambari-server/src/main/java/org/apache/ambari/server/agent/HeartBeatHandler.java
>  0f284a2 
>   
> ambari-server/src/main/java/org/apache/ambari/server/api/services/AmbariMetaInfo.java
>  bafd9b5 
>   
> ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java
>  93b0dca 
>   
> ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostComponentDesiredStateEntity.java
>  8b29852 
>   
> ambari-server/src/main/java/org/apache/ambari/server/state/ConfigHelper.java 
> 964d2d6 
>   
> ambari-server/src/main/java/org/apache/ambari/server/state/ServiceComponentHost.java
>  0938af6 
>   ambari-server/src/main/java/org/apache/ambari/server/state/ServiceInfo.java 
> 39566d9 
>   
> ambari-server/src/main/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostImpl.java
>  1d4ae4b 
>   
> ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog160.java
>  f66fe1c 
>   ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 1d04d0b 
>   ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql ae18892 
>   ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql 1be43f1 
>   
> ambari-server/src/main/resources/stacks/HDP/1.3.2/services/GANGLIA/metainfo.xml
>  528db40 
>   
> ambari-server/src/main/resources/stacks/HDP/1.3.2/services/NAGIOS/metainfo.xml
>  74f6c2b 
>   
> ambari-server/src/main/resources/stacks/HDP/2.0.6/services/GANGLIA/metainfo.xml
>  98703bc 
>   
> ambari-server/src/main/resources/stacks/HDP/2.0.6/services/NAGIOS/metainfo.xml
>  f435f6d 
>   
> ambari-server/src/test/java/org/apache/ambari/server/agent/TestHeartbeatHandler.java
>  f0b130b 
>   
> ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariManagementControllerTest.java
>  9727db9 
>   
> ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog160Test.java
>  e2234b2 
>   
> ambari-server/src/test/resources/stacks/HDP/2.0.5/services/NAGIOS/metainfo.xml
>  76471cf 
> 
> Diff: https://reviews.apache.org/r/20263/diff/
> 
> 
> Testing
> -------
> 
> All service tests passed
> 
> + Tested manually with Postgres, Mysql, Oracle as Ambari DB, because DB 
> schema was updated.
> 
> 
> File Attachments
> ----------------
> 
> AMBARI-5445.patch
>   
> https://reviews.apache.org/media/uploaded/files/2014/04/18/9b13ab26-436d-475d-acd1-f44fa55904a3__AMBARI-5445.patch
> 
> 
> Thanks,
> 
> Dmytro Sen
> 
>

Reply via email to