Mercurial > dovecot > core-2.2

changeset 22414:7ee97c9d9b56

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
fts: Log when indexing requires adding more mails to index than requested This should mainly happen when FTS indexing is requested for a virtual mailbox.
author Timo Sirainen <timo.sirainen@dovecot.fi>
date Sun, 18 Jun 2017 13:07:14 +0300
parents 633ca13ae99c
children ebeda476561b
files src/plugins/fts/fts-storage.c
diffstat 1 files changed, 14 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/src/plugins/fts/fts-storage.c	Sun Jun 18 12:18:31 2017 +0300
+++ b/src/plugins/fts/fts-storage.c	Sun Jun 18 13:07:14 2017 +0300
@@ -52,6 +52,7 @@
 	struct fts_scores *scores;
 	uint32_t next_index_seq;
 	uint32_t highest_virtual_uid;
+	unsigned int precache_extra_count;
 
 	unsigned int precached:1;
 	unsigned int mails_saved:1;
@@ -430,7 +431,7 @@
 static int
 fts_mail_precache_range(struct mailbox_transaction_context *trans,
 			struct fts_backend_update_context *update_ctx,
-			uint32_t seq1, uint32_t seq2)
+			uint32_t seq1, uint32_t seq2, unsigned int *extra_count)
 {
 	struct mail_search_args *search_args;
 	struct mail_search_context *ctx;
@@ -451,6 +452,7 @@
 			break;
 		}
 		mail_precache(mail);
+		*extra_count += 1;
 	}
 	if (mailbox_search_deinit(&ctx) < 0)
 		ret = -1;
@@ -495,7 +497,8 @@
 		if (fts_mail_precache_range(_mail->transaction,
 					    flist->update_ctx,
 					    ft->next_index_seq,
-					    _mail->seq-1) < 0) {
+					    _mail->seq-1,
+					    &ft->precache_extra_count) < 0) {
 			ft->failed = TRUE;
 			return;
 		}
@@ -586,6 +589,15 @@
 	}
 	if (ft->scores != NULL)
 		fts_scores_unref(&ft->scores);
+	if (ft->precache_extra_count > 0) {
+		if (ret < 0) {
+			i_error("fts: Failed after indexing %u extra mails internally in %s: %s",
+			       ft->precache_extra_count, t->box->vname, *error_r);
+		} else {
+			i_info("fts: Indexed %u extra mails internally in %s",
+			       ft->precache_extra_count, t->box->vname);
+		}
+	}
 	i_free(ft);
 	return ret;
 }