Change log file names to use toISOString() rather than millis

benschmaus · Jul 29, 2011 · d0a7b4e · d0a7b4e
1 parent d4893f8
commit d0a7b4e
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 4 deletions.
diff --git a/RELEASE-NOTES.md b/RELEASE-NOTES.md
@@ -5,6 +5,7 @@ Compatible with node v0.4.x
 Features:
 
 * Add graphJmx() and graphProcess(); deprecate spawnAndMonitor(). These provide an easy way to graph JMX attributes as well as output from external processes, such as iostat.
+* More readable date strings are used in log files names
 * rps is a separate stat from result codes
 * Test Results page timestamp does not auto-update on open
 * X-axes now use real timestamps rather than minutes since test start

diff --git a/lib/monitoring/statslogger.js b/lib/monitoring/statslogger.js
@@ -9,7 +9,7 @@ var LogFile = require('../stats').LogFile;
 
 /** StatsLogger writes interval stats from a Monitor or MonitorGroup to disk each time it emits 'update' */
 var StatsLogger = exports.StatsLogger = function StatsLogger(monitor, logNameOrObject) {
-    this.logNameOrObject = logNameOrObject || ('results-' + START.getTime() + '-stats.log');
+    this.logNameOrObject = logNameOrObject || ('results-' + START.toISOString() + '-stats.log');
     this.monitor = monitor;
     this.logger_ = this.log_.bind(this);
 };

diff --git a/lib/reporting/report.js b/lib/reporting/report.js
@@ -126,7 +126,7 @@ Chart.prototype = {
 
 var ReportGroup = exports.ReportGroup = function() {
     this.reports = [];
-    this.logNameOrObject = 'results-' + START.getTime() + '.html';
+    this.logNameOrObject = 'results-' + START.toISOString() + '.html';
 };
 ReportGroup.prototype = {
     addReport: function(report) {

diff --git a/nodeload.js b/nodeload.js