ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alejandro Fernandez" <afernan...@hortonworks.com>
Subject Re: Review Request 26847: Events with Service Checks results in Exceptions in log
Date Fri, 17 Oct 2014 01:00:49 GMT

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



ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java
<https://reviews.apache.org/r/26847/#comment97570>

    This may need a null check before calling getName()


- Alejandro Fernandez


On Oct. 17, 2014, 12:51 a.m., Sid Wagle wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/26847/
> -----------------------------------------------------------
> 
> (Updated Oct. 17, 2014, 12:51 a.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez and Jonathan Hurley.
> 
> 
> Bugs: AMBARI-7822
>     https://issues.apache.org/jira/browse/AMBARI-7822
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Component is null since it is a service check action.
> 
> 14:33:08,558 ERROR [Thread-21] ClusterImpl:1452 - ServiceComponentHost lookup exception
> org.apache.ambari.server.ServiceComponentNotFoundException: ServiceComponent not found,
clusterName=cl1, serviceName=WEBHCAT, serviceComponentName=null
>        at org.apache.ambari.server.state.ServiceImpl.getServiceComponent(ServiceImpl.java:279)
>        at org.apache.ambari.server.state.cluster.ClusterImpl.processServiceComponentHostEvents(ClusterImpl.java:1448)
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java
d12b6fd 
>   ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariManagementControllerImplTest.java
affa4c9 
> 
> Diff: https://reviews.apache.org/r/26847/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Sid Wagle
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message