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

Review request for Ambari, Sid Wagle and Tom Beerbower.


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


Repository: ambari


Description
-------

* Hive service/component name not taken into account
* Hive status check puts the whole stack trace, leaving key information at the 
end


Diffs
-----

  
ambari-server/src/main/java/org/apache/ambari/server/controller/nagios/NagiosPropertyProvider.java
 a57c87b 
  
ambari-server/src/main/resources/stacks/HDP/1.3.2/services/NAGIOS/package/files/check_wrapper.sh
 ece44b2 
  
ambari-server/src/main/resources/stacks/HDP/2.0.6/services/NAGIOS/package/files/check_wrapper.sh
 a1b6a12 
  
ambari-server/src/test/java/org/apache/ambari/server/controller/nagios/NagiosPropertyProviderTest.java
 209e04a 
  ambari-server/src/test/resources/nagios_alerts.txt a90f445 

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


Testing
-------

Added test case for "complex" Hive error output.  No tests for check_wrapper.sh 
(we don't have any tests for Nagios plugins)


Thanks,

Nate Cole

Reply via email to