summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMatt A. Tobin <email@mattatobin.com>2019-11-10 18:53:43 -0500
committerMatt A. Tobin <email@mattatobin.com>2019-11-10 18:53:43 -0500
commit4db1ae892bad565e8e59ec6034b4c98946077248 (patch)
treef515b362f2b3f3b42f7afc231028176ebebdadcf
parente60090bc9c7e14bb8253eeb64658aedaa0f863c7 (diff)
downloadUXP-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.js11
-rw-r--r--mailnews/db/gloda/modules/index_msg.js1
-rw-r--r--mailnews/db/gloda/modules/indexer.js19
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;