-----------------------------------------------------------
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