ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Nate Cole" <nc...@hortonworks.com>
Subject Review Request 24064: Flume: API required to provide temporal min/max/avg/sum/rate for a Flume metric
Date Tue, 29 Jul 2014 20:24:04 GMT

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

Review request for Ambari, Sid Wagle and Tom Beerbower.


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


Repository: ambari


Description
-------

Updated rrd.py to do the following use case:  Collect metrics across regex-matching rrd files
and provide min/max/avg/sum aggregate metrics across all hosts.  Flume requires special metrics
considering many processes can coexist on one host.

Also addressed a small issue that was forcing the UI to show duplicate alerts.  I just threw
that in because I had noticed it.


Diffs
-----

  ambari-server/src/main/java/org/apache/ambari/server/controller/nagios/NagiosPropertyProvider.java
cc0f939 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/FLUME/metrics.json c560d2a 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/FLUME/package/scripts/flume.py
1cbdf39 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/GANGLIA/package/templates/rrd.py.j2
a3561e9 
  ambari-server/src/test/java/org/apache/ambari/server/controller/nagios/NagiosPropertyProviderTest.java
cbd3262 
  ambari-server/src/test/python/stacks/2.0.6/FLUME/test_flume.py 1abd849 

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


Testing
-------

[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 19:56.017s
[INFO] Finished at: Tue Jul 29 16:22:21 EDT 2014
[INFO] Final Memory: 21M/123M
[INFO] ------------------------------------------------------------------------


Thanks,

Nate Cole


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