From f5a00a8bc68cb246bfe9265e7f66856db98f5f7d Mon Sep 17 00:00:00 2001 From: Timo Sirainen Date: Mon, 5 Feb 2018 22:21:13 +0200 Subject: [PATCH] fts: Don't reindex FTS mails if .cache file is deleted This means that if fts is enabled, "doveadm index" no longer adds mails to dovecot.index.cache if it's deleted. However, it was rarely used for that purpose. More likely due to a corrupted cache file all the mails were unnecessarily being opened and reindexed. --- src/plugins/fts/fts-storage.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/plugins/fts/fts-storage.c b/src/plugins/fts/fts-storage.c index 4b68663097..deda167854 100644 --- a/src/plugins/fts/fts-storage.c +++ b/src/plugins/fts/fts-storage.c @@ -109,9 +109,10 @@ fts_mailbox_get_status(struct mailbox *box, enum mailbox_status_items items, if (fts_mailbox_get_last_cached_seq(box, &seq) < 0) return -1; - /* use whichever is smaller */ - if (status_r->last_cached_seq > seq) - status_r->last_cached_seq = seq; + /* Always use the FTS's last_cached_seq. This is because we + don't want to reindex all mails to FTS if .cache file is + deleted. */ + status_r->last_cached_seq = seq; } return 0; }