Add an error parameter to the stats write callback.

puntofisso · Apr 26, 2012 · 168a8af · 168a8af
1 parent a376106
commit 168a8af
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/backends/graphite.js b/backends/graphite.js
@@ -121,7 +121,7 @@ var flush_stats = function(ts, flushInterval, metrics) {
 
 var write_stats = function(writeCb) {
   for (stat in graphiteStats) {
-    writeCb(stat, graphiteStats[stat]);
+    writeCb(null, stat, graphiteStats[stat]);
   }
 };
 

diff --git a/stats.js b/stats.js
@@ -176,8 +176,13 @@ config.configFile(process.argv[2], function (config, oldConfig) {
 
             // Retrieve stats from each backend
             for (var i = 0; i < backends.length; i++) {
-              backends[i].mod.write_stats(function(stat, val) {
-                stat_writer(backends[i].name, stat, val);
+              backends[i].mod.write_stats(function(err, stat, val) {
+                if (err) {
+                  util.log("Failed to read stats for backend " +
+                           backends[i].name + ": " + err);
+                } else {
+                  stat_writer(backends[i].name, stat, val);
+                }
               });
             }