From 4db1ae892bad565e8e59ec6034b4c98946077248 Mon Sep 17 00:00:00 2001 From: "Matt A. Tobin" Date: Sun, 10 Nov 2019 18:53:43 -0500 Subject: Issue #1273 - Remove Telemetry from Gloda --- mailnews/db/gloda/modules/datastore.js | 11 ----------- mailnews/db/gloda/modules/index_msg.js | 1 - mailnews/db/gloda/modules/indexer.js | 19 ------------------- 3 files changed, 31 deletions(-) (limited to 'mailnews/db/gloda/modules') diff --git a/mailnews/db/gloda/modules/datastore.js b/mailnews/db/gloda/modules/datastore.js index 70bbdc6a7..c9374c563 100644 --- a/mailnews/db/gloda/modules/datastore.js +++ b/mailnews/db/gloda/modules/datastore.js @@ -1030,17 +1030,6 @@ var GlodaDatastore = { var dbConnection; - // Report about the size of the database through telemetry (if there's a - // database, naturally). - if (dbFile.exists()) { - try { - let h = Services.telemetry.getHistogramById("THUNDERBIRD_GLODA_SIZE_MB"); - h.add(dbFile.fileSize/1048576); - } catch (e) { - this._log.warn("Couldn't report telemetry", e); - } - } - // Create the file if it does not exist if (!dbFile.exists()) { this._log.debug("Creating database because it doesn't exist."); diff --git a/mailnews/db/gloda/modules/index_msg.js b/mailnews/db/gloda/modules/index_msg.js index d4d7a8ceb..8aec476a3 100644 --- a/mailnews/db/gloda/modules/index_msg.js +++ b/mailnews/db/gloda/modules/index_msg.js @@ -1460,7 +1460,6 @@ var GlodaMsgIndexer = { yield aCallbackHandle.pushAndGo( this._indexMessage(msgHdr, aCallbackHandle), {what: "indexMessage", msgHdr: msgHdr}); - GlodaIndexer._indexedMessageCount++; if (logDebug) this._log.debug("<<< back from _indexMessage"); } diff --git a/mailnews/db/gloda/modules/indexer.js b/mailnews/db/gloda/modules/indexer.js index f6c939530..e75b6796b 100644 --- a/mailnews/db/gloda/modules/indexer.js +++ b/mailnews/db/gloda/modules/indexer.js @@ -1006,10 +1006,6 @@ var GlodaIndexer = { // make ourselves less responsive by drawing out the period of time we // are dominating the main thread. this._perfIndexStopwatch.start(); - // For telemetry purposes, we want to know how many messages we've been - // processing during that batch, and how long it took, pauses included. - let t0 = Date.now(); - this._indexedMessageCount = 0; batchCount = 0; while (batchCount < this._indexTokens) { if ((this._callbackHandle.activeIterator === null) && @@ -1150,21 +1146,6 @@ var GlodaIndexer = { } } - // All pauses have been taken, how effective were we? Report! - // XXX: there's possibly a lot of fluctuation since we go through here - // every 5 messages or even less - if (this._indexedMessageCount > 0) { - let delta = (Date.now() - t0)/1000; // in seconds - let v = Math.round(this._indexedMessageCount/delta); - try { - let h = Services.telemetry - .getHistogramById("THUNDERBIRD_INDEXING_RATE_MSG_PER_S"); - h.add(v); - } catch (e) { - this._log.warn("Couldn't report telemetry", e, v); - } - } - if (batchCount > 0) { let totalTime = this._perfIndexStopwatch.realTimeSeconds * 1000; let timePerToken = totalTime / batchCount; -- cgit v1.2.3