diff options
author | Matt A. Tobin <email@mattatobin.com> | 2019-11-10 18:53:43 -0500 |
---|---|---|
committer | Matt A. Tobin <email@mattatobin.com> | 2019-11-10 18:53:43 -0500 |
commit | 4db1ae892bad565e8e59ec6034b4c98946077248 (patch) | |
tree | f515b362f2b3f3b42f7afc231028176ebebdadcf | |
parent | e60090bc9c7e14bb8253eeb64658aedaa0f863c7 (diff) | |
download | UXP-4db1ae892bad565e8e59ec6034b4c98946077248.tar UXP-4db1ae892bad565e8e59ec6034b4c98946077248.tar.gz UXP-4db1ae892bad565e8e59ec6034b4c98946077248.tar.lz UXP-4db1ae892bad565e8e59ec6034b4c98946077248.tar.xz UXP-4db1ae892bad565e8e59ec6034b4c98946077248.zip |
Issue #1273 - Remove Telemetry from Gloda
-rw-r--r-- | mailnews/db/gloda/modules/datastore.js | 11 | ||||
-rw-r--r-- | mailnews/db/gloda/modules/index_msg.js | 1 | ||||
-rw-r--r-- | mailnews/db/gloda/modules/indexer.js | 19 |
3 files changed, 0 insertions, 31 deletions
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; |