-/*
- * This module handles fulltext indexing of the message base.
- * Copyright (c) 2005-2022 by the citadel.org team
- *
- * This program is open source software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License as published
- * by the Free Software Foundation; either version 3 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
- */
+// This module handles fulltext indexing of the message base.
+//
+// Copyright (c) 2005-2023 by the citadel.org team
+//
+// This program is open source software. Use, duplication, or disclosure
+// is subject to the terms of the GNU General Public License, version 3.
#include "../../sysdep.h"
#include <stdlib.h>
#include <string.h>
#include <limits.h>
#include <libcitadel.h>
-#include "../../citadel.h"
+#include "../../citadel_defs.h"
#include "../../server.h"
#include "../../citserver.h"
#include "../../support.h"
#include "../../config.h"
+#include "../../room_ops.h"
#include "../../database.h"
#include "../../msgbase.h"
#include "../../control.h"
#include "../../context.h"
#include "../../ctdl_module.h"
-long ft_newhighest = 0L;
-long *ft_newmsgs = NULL;
-int ft_num_msgs = 0;
-int ft_num_alloc = 0;
+// These can be global variables because only one indexer runs at a time.
+Array *messages_to_be_indexed = NULL;
+long highest_msg_already_indexed = 0;
+long highest_msg_to_be_indexed = 0;
-int ftc_num_msgs[65536];
-long *ftc_msgs[65536];
-
-/*
- * Compare function
- */
+// Compare function
int longcmp(const void *rec1, const void *rec2) {
long i1, i2;
}
-/*
- * Flush our index cache out to disk.
- */
-void ft_flush_cache(void) {
- int i;
- time_t last_update = 0;
-
- for (i=0; i<65536; ++i) {
- if ((time(NULL) - last_update) >= 10) {
- syslog(LOG_INFO,
- "fulltext: flushing index cache to disk (%d%% complete)",
- (i * 100 / 65536)
- );
- last_update = time(NULL);
- }
- if (ftc_msgs[i] != NULL) {
- cdb_store(CDB_FULLTEXT, &i, sizeof(int), ftc_msgs[i],
- (ftc_num_msgs[i] * sizeof(long)));
- ftc_num_msgs[i] = 0;
- free(ftc_msgs[i]);
- ftc_msgs[i] = NULL;
- }
- }
- syslog(LOG_INFO, "fulltext: flushed index cache to disk (100%% complete)");
-}
-
-
-/*
- * Index or de-index a message. (op == 1 to index, 0 to de-index)
- */
+// Index or de-index a message. (op == 1 to index, 0 to de-index)
void ft_index_message(long msgnum, int op) {
- int num_tokens = 0;
- int *tokens = NULL;
int i, j;
- struct cdbdata *cdb_bucket;
+ Array *tokens_in_this_message = NULL;
+ struct cdbdata cdb_bucket;
StrBuf *msgtext;
char *txt;
int tok;
struct CtdlMessage *msg = NULL;
+ if (msgnum == 0) return;
+
msg = CtdlFetchMessage(msgnum, 1);
if (msg == NULL) {
+ // This is not necessarily an error condition; it could simply mean that the message was
+ // deleted before it could be indexed. This happens often when the load tester is running.
syslog(LOG_ERR, "fulltext: ft_index_message() could not load msg %ld", msgnum);
return;
}
syslog(LOG_DEBUG, "fulltext: ft_index_message() %s msg %ld", (op ? "adding" : "removing") , msgnum);
- /* Output the message as text before indexing it, so we don't end up
- * indexing a bunch of encoded base64, etc.
- */
+ // Output the message as text before indexing it, so we don't end up indexing a bunch of encoded base64, etc.
CC->redirect_buffer = NewStrBufPlain(NULL, SIZ);
CtdlOutputPreLoadedMsg(msg, MT_CITADEL, HEADERS_ALL, 0, 1, 0);
CM_Free(msg);
syslog(LOG_DEBUG, "fulltext: wordbreaking message %ld (%d bytes)", msgnum, StrLength(msgtext));
}
txt = SmashStrBuf(&msgtext);
- wordbreaker(txt, &num_tokens, &tokens);
+ tokens_in_this_message = wordbreaker(txt);
free(txt);
- syslog(LOG_DEBUG, "fulltext: indexing message %ld [%d tokens]", msgnum, num_tokens);
- if (num_tokens > 0) {
- for (i=0; i<num_tokens; ++i) {
+ syslog(LOG_DEBUG, "fulltext: %sindexing message %ld [%d tokens]",
+ (op ? "" : "de"),
+ msgnum,
+ array_len(tokens_in_this_message)
+ );
- /* Add the message to the relevant token bucket */
+ if (tokens_in_this_message == NULL) {
+ return;
+ }
- /* search for tokens[i] */
- tok = tokens[i];
+ if (array_len(tokens_in_this_message) > 0) {
+ begin_critical_section(S_INDEXER);
+ cdb_begin_transaction();
+ for (i=0; i<array_len(tokens_in_this_message); ++i) {
- if ( (tok >= 0) && (tok <= 65535) ) {
- /* fetch the bucket, Liza */
- if (ftc_msgs[tok] == NULL) {
- cdb_bucket = cdb_fetch(CDB_FULLTEXT, &tok, sizeof(int));
- if (cdb_bucket != NULL) {
- ftc_num_msgs[tok] = cdb_bucket->len / sizeof(long);
- ftc_msgs[tok] = (long *)cdb_bucket->ptr;
- cdb_bucket->ptr = NULL;
- cdb_free(cdb_bucket);
- }
- else {
- ftc_num_msgs[tok] = 0;
- ftc_msgs[tok] = malloc(sizeof(long));
+ // Identify the bucket which we will be modifying
+ memcpy(&tok, array_get_element_at(tokens_in_this_message, i), sizeof(int));
+
+ // fetch the bucket
+ cdb_bucket = cdb_fetch(CDB_FULLTEXT, &tok, sizeof(int));
+ long *newbucket = malloc(cdb_bucket.len + sizeof(long));
+ int nmsgs = cdb_bucket.len / sizeof(long);
+
+ if (op == 1) { // indexing, add this message to the bucket
+ memcpy(&newbucket[0], cdb_bucket.ptr, cdb_bucket.len);
+ int already_there = 0;
+ for (j=0; j<nmsgs; ++j) {
+ if (newbucket[j] == msgnum) {
+ already_there = 1;
}
}
-
-
- if (op == 1) { /* add to index */
- ++ftc_num_msgs[tok];
- ftc_msgs[tok] = realloc(ftc_msgs[tok],
- ftc_num_msgs[tok]*sizeof(long));
- ftc_msgs[tok][ftc_num_msgs[tok] - 1] = msgnum;
+ if (already_there == 0) {
+ memcpy(&newbucket[nmsgs++], &msgnum, sizeof(long));
}
-
- if (op == 0) { /* remove from index */
- if (ftc_num_msgs[tok] >= 1) {
- for (j=0; j<ftc_num_msgs[tok]; ++j) {
- if (ftc_msgs[tok][j] == msgnum) {
- memmove(&ftc_msgs[tok][j], &ftc_msgs[tok][j+1], ((ftc_num_msgs[tok] - j - 1)*sizeof(long)));
- --ftc_num_msgs[tok];
- --j;
- }
- }
+ }
+
+ else if (op == 0) { // deindexing, remove this message from the bucket
+ memcpy(newbucket, cdb_bucket.ptr, cdb_bucket.len);
+ for (j=0; j<nmsgs; ++j) {
+ if ((newbucket[j] == msgnum) || (newbucket[j] == 0)) {
+ memcpy(&newbucket[j], &newbucket[j+1], ((nmsgs-j)*sizeof(long)));
+ --j;
+ --nmsgs;
}
}
}
- else {
- syslog(LOG_ALERT, "fulltext: invalid token %d !!", tok);
- }
- }
- free(tokens);
+ // Then write it back to disk
+ cdb_store(CDB_FULLTEXT, &tok, sizeof(int), newbucket, (nmsgs*sizeof(long)));
+ free(newbucket);
+
+ if (server_shutting_down) break;
+ }
+ cdb_end_transaction();
+ end_critical_section(S_INDEXER);
+ CtdlSetConfigLong("MMfulltext", msgnum);
}
+ array_free(tokens_in_this_message);
}
-/*
- * Add a message to the list of those to be indexed.
- */
-void ft_index_msg(long msgnum, void *userdata) {
-
- if ((msgnum > CtdlGetConfigLong("MMfulltext")) && (msgnum <= ft_newhighest)) {
- ++ft_num_msgs;
- if (ft_num_msgs > ft_num_alloc) {
- ft_num_alloc += 1024;
- ft_newmsgs = realloc(ft_newmsgs, (ft_num_alloc * sizeof(long)));
- }
- ft_newmsgs[ft_num_msgs - 1] = msgnum;
+// Scan a room for messages to index.
+void ft_index_room(struct ctdlroom *qrbuf, void *data) {
+ if (server_shutting_down) {
+ return;
}
-}
+ int num_msgs = 0;
+ long *msglist;
+ int i;
+ // 2023aug30 ajc - old code did another CtdlGetRoom() here. Not only is that redundant,
+ // but for some reason it also made Berkeley DB deadlock after a while. I don't know why.
-/*
- * Scan a room for messages to index.
- */
-void ft_index_room(struct ctdlroom *qrbuf, void *data)
-{
- if (server_shutting_down)
- return;
-
- CtdlGetRoom(&CC->room, qrbuf->QRname);
- CtdlForEachMessage(MSGS_ALL, 0L, NULL, NULL, NULL, ft_index_msg, NULL);
+ // qrbuf is already populated. fetch the list of messages in this room.
+ num_msgs = CtdlFetchMsgList(qrbuf->QRnumber, &msglist);
+
+ // Identify messages which have NOT yet been seen by the indexer.
+ if (msglist != NULL) {
+ for (i=0; i<num_msgs; ++i) {
+ if (
+ (msglist[i] > 0)
+ && (msglist[i] > highest_msg_already_indexed)
+ && (msglist[i] <= highest_msg_to_be_indexed)
+ ) {
+ array_append(messages_to_be_indexed, &msglist[i]);
+ }
+ }
+ free(msglist);
+ }
}
-/*
- * Begin the fulltext indexing process.
- */
+// Begin the fulltext indexing process.
void do_fulltext_indexing(void) {
int i;
static time_t last_progress = 0L;
static int is_running = 0;
- if (is_running) return; /* Concurrency check - only one can run */
+
+ if (is_running) return; // Concurrency check - only one can run
is_running = 1;
- /*
- * Don't do this if the site doesn't have it enabled.
- */
+ // Don't do this if the site doesn't have it enabled.
if (!CtdlGetConfigInt("c_enable_fulltext")) {
+ is_running = 0;
return;
}
-
- /*
- * If we've switched wordbreaker modules, burn the index and start over.
- */
+ // If we've switched wordbreaker modules, burn the index and start over.
begin_critical_section(S_CONTROL);
if (CtdlGetConfigInt("MM_fulltext_wordbreaker") != FT_WORDBREAKER_ID) {
syslog(LOG_DEBUG, "fulltext: wb ver on disk = %d, code ver = %d",
}
end_critical_section(S_CONTROL);
- /*
- * Silently return if our fulltext index is up to date with new messages.
- */
+ // Silently return if our fulltext index is up to date with new messages.
if ((CtdlGetConfigLong("MMfulltext") >= CtdlGetConfigLong("MMhighest"))) {
- return; /* nothing to do! */
+ is_running = 0;
+ return; // nothing to do!
}
- /*
- * Now go through each room and find messages to index.
- */
- ft_newhighest = CtdlGetConfigLong("MMhighest");
- CtdlForEachRoom(ft_index_room, NULL); /* load all msg pointers */
-
- if (ft_num_msgs > 0) {
- qsort(ft_newmsgs, ft_num_msgs, sizeof(long), longcmp);
- for (i=0; i<(ft_num_msgs-1); ++i) { /* purge dups */
- if (ft_newmsgs[i] == ft_newmsgs[i+1]) {
- memmove(&ft_newmsgs[i], &ft_newmsgs[i+1],
- ((ft_num_msgs - i - 1)*sizeof(long)));
- --ft_num_msgs;
- --i;
- }
- }
+ highest_msg_already_indexed = CtdlGetConfigLong("MMfulltext");
+ highest_msg_to_be_indexed = CtdlGetConfigLong("MMhighest");
+ syslog(LOG_DEBUG, "fulltext: indexing started. msgs %ld--%ld", highest_msg_already_indexed, highest_msg_to_be_indexed);
- /* Here it is ... do each message! */
- for (i=0; i<ft_num_msgs; ++i) {
- if (time(NULL) != last_progress) {
- syslog(LOG_DEBUG,
- "fulltext: indexed %d of %d messages (%d%%)",
- i, ft_num_msgs,
- ((i*100) / ft_num_msgs)
- );
- last_progress = time(NULL);
- }
- ft_index_message(ft_newmsgs[i], 1);
+ messages_to_be_indexed = array_new(sizeof(long));
- /* Check to see if we need to quit early */
- if (server_shutting_down) {
- syslog(LOG_DEBUG, "fulltext: indexer quitting early");
- ft_newhighest = ft_newmsgs[i];
- break;
- }
+ // Now go through each room and find messages to index.
+ CtdlForEachRoom(ft_index_room, NULL); // load all msg pointers
+ array_sort(messages_to_be_indexed, longcmp); // sort them
- /* Check to see if we have to maybe flush to disk */
- if (i >= FT_MAX_CACHE) {
- syslog(LOG_DEBUG, "fulltext: time to flush.");
- ft_newhighest = ft_newmsgs[i];
- break;
- }
+ // Here it is ... do each message!
+ long msgnum = 0;
+ long prev_msgnum = 0;
+ time_t started_indexing_at = time(NULL);
+ int yielded = 0;
+ for (i=0; ((i<array_len(messages_to_be_indexed)) && (yielded == 0)); ++i) {
+ memcpy(&msgnum, array_get_element_at(messages_to_be_indexed, i), sizeof(long));
+ if (msgnum != prev_msgnum) { // careful to avoid dupes
+ ft_index_message(msgnum, 1);
+ }
+ prev_msgnum = msgnum;
+
+ // If we run too long, yield the thread so the server can do other things. We'll be back.
+ if (time(NULL) - started_indexing_at >= MAXIMUM_INDEXER_RUN_TIME) {
+ yielded = 1;
}
-
- free(ft_newmsgs);
- ft_num_msgs = 0;
- ft_num_alloc = 0;
- ft_newmsgs = NULL;
}
- if (server_shutting_down) {
- is_running = 0;
- return;
- }
-
- /* Save our place so we don't have to do this again */
- ft_flush_cache();
- begin_critical_section(S_CONTROL);
- CtdlSetConfigLong("MMfulltext", ft_newhighest);
+ array_free(messages_to_be_indexed);
CtdlSetConfigInt("MM_fulltext_wordbreaker", FT_WORDBREAKER_ID);
- end_critical_section(S_CONTROL);
+ syslog(LOG_DEBUG, "fulltext: indexer has run for %ld seconds; yielded=%d", time(NULL) - started_indexing_at, yielded);
+
+ // This keeps the indexer from starting up over and over if the highest message in the list was deleted
+ // before we had a chance to index it.
+ if (!yielded) {
+ CtdlSetConfigLong("MMfulltext", highest_msg_to_be_indexed);
+ }
- syslog(LOG_DEBUG, "fulltext: indexing finished");
is_running = 0;
return;
}
-/*
- * API call to perform searches.
- * (This one does the "all of these words" search.)
- * Caller is responsible for freeing the message list.
- */
-void ft_search(int *fts_num_msgs, long **fts_msgs, const char *search_string) {
- int num_tokens = 0;
- int *tokens = NULL;
- int i, j;
- struct cdbdata *cdb_bucket;
- int num_all_msgs = 0;
- long *all_msgs = NULL;
- int num_ret_msgs = 0;
- int num_ret_alloc = 0;
- long *ret_msgs = NULL;
- int tok;
-
- wordbreaker(search_string, &num_tokens, &tokens);
- if (num_tokens > 0) {
- for (i=0; i<num_tokens; ++i) {
-
- /* search for tokens[i] */
- tok = tokens[i];
-
- /* fetch the bucket, Liza */
- if (ftc_msgs[tok] == NULL) {
- cdb_bucket = cdb_fetch(CDB_FULLTEXT, &tok, sizeof(int));
- if (cdb_bucket != NULL) {
- ftc_num_msgs[tok] = cdb_bucket->len / sizeof(long);
- ftc_msgs[tok] = (long *)cdb_bucket->ptr;
- cdb_bucket->ptr = NULL;
- cdb_free(cdb_bucket);
- }
- else {
- ftc_num_msgs[tok] = 0;
- ftc_msgs[tok] = malloc(sizeof(long));
+// API call to perform searches.
+// (This one does the "all of these words" search.)
+// Caller is responsible for freeing the message list.
+Array *CtdlFullTextSearch(const char *search_string) {
+ int i, j, tok;
+ struct cdbdata cdb_bucket;
+ long msgnum, smsgnum;
+ int count;
+
+ Array *r = array_new(sizeof(long));
+ if (!r) return(NULL);
+ Array *t = wordbreaker(search_string);
+
+ if ((t != NULL) && (array_len(t) > 0)) {
+ for (i=0; i<array_len(t); ++i) {
+ memcpy(&tok, array_get_element_at(t, i), sizeof(int));
+ cdb_bucket = cdb_fetch(CDB_FULLTEXT, &tok, sizeof(int));
+ if (cdb_bucket.ptr != NULL) {
+ for (j=0; j<(cdb_bucket.len / sizeof(long)); ++j) {
+ memcpy(&msgnum, cdb_bucket.ptr + (j*sizeof(long)), sizeof(long));
+ array_append(r, &msgnum);
}
}
+ }
+ }
- num_all_msgs += ftc_num_msgs[tok];
- if (num_all_msgs > 0) {
- all_msgs = realloc(all_msgs, num_all_msgs*sizeof(long) );
- memcpy(&all_msgs[num_all_msgs-ftc_num_msgs[tok]],
- ftc_msgs[tok], ftc_num_msgs[tok]*sizeof(long) );
+ // We need to return any message containing ALL of the tokens.
+ // If a message number appears in the array `n` times, where `n` is the number of search words,
+ // that means it matched all search words.
+ array_sort(r, longcmp);
+ for (i=0; i<array_len(r); ++i) {
+ memcpy(&msgnum, array_get_element_at(r, i), sizeof(long));
+ count = 1;
+ for (j=i+1; j<array_len(r); ++j) {
+ memcpy(&smsgnum, array_get_element_at(r, j), sizeof(long));
+ if (msgnum == smsgnum) {
+ ++count;
+ array_delete_element_at(r, j);
+ --j;
}
-
}
- free(tokens);
- if (all_msgs != NULL) {
- qsort(all_msgs, num_all_msgs, sizeof(long), longcmp);
-
- /*
- * At this point, if a message appears num_tokens times in the
- * list, then it contains all of the search tokens.
- */
- if (num_all_msgs >= num_tokens)
- for (j=0; j<(num_all_msgs-num_tokens+1); ++j) {
- if (all_msgs[j] == all_msgs[j+num_tokens-1]) {
-
- ++num_ret_msgs;
- if (num_ret_msgs > num_ret_alloc) {
- num_ret_alloc += 64;
- ret_msgs = realloc(ret_msgs,
- (num_ret_alloc*sizeof(long)) );
- }
- ret_msgs[num_ret_msgs - 1] = all_msgs[j];
-
- }
- }
- free(all_msgs);
+ if (count != array_len(t)) {
+ array_delete_element_at(r, i);
+ --i;
}
}
- *fts_num_msgs = num_ret_msgs;
- *fts_msgs = ret_msgs;
+ array_free(t);
+ return(r);
}
-/*
- * This search command is for diagnostic purposes and may be removed or replaced.
- */
+// This search command is for diagnostic purposes and may be removed or replaced.
void cmd_srch(char *argbuf) {
- int num_msgs = 0;
- long *msgs = NULL;
int i;
- char search_string[256];
+ char search_string[SIZ];
+ Array *matches = NULL;
+ long msgnum;
if (CtdlAccessCheck(ac_logged_in)) return;
if (!CtdlGetConfigInt("c_enable_fulltext")) {
- cprintf("%d Full text index is not enabled on this server.\n",
- ERROR + CMD_NOT_SUPPORTED);
+ cprintf("%d Full text index is not enabled on this server.\n", ERROR + CMD_NOT_SUPPORTED);
return;
}
extract_token(search_string, argbuf, 0, '|', sizeof search_string);
- ft_search(&num_msgs, &msgs, search_string);
+ matches = CtdlFullTextSearch(search_string);
- cprintf("%d %d msgs match all search words:\n",
- LISTING_FOLLOWS, num_msgs);
- if (num_msgs > 0) {
- for (i=0; i<num_msgs; ++i) {
- cprintf("%ld\n", msgs[i]);
+ cprintf("%d %d msgs match all search words:\n", LISTING_FOLLOWS, array_len(matches));
+ if ((matches != NULL) && (array_len(matches) > 0)) {
+ for (i=0; i<array_len(matches); ++i) {
+ memcpy(&msgnum, array_get_element_at(matches, i), sizeof(long));
+ cprintf("%ld\n", msgnum);
}
}
- if (msgs != NULL) free(msgs);
+ if (matches != NULL) {
+ array_free(matches);
+ }
cprintf("000\n");
}
-/*
- * Zero out our index cache.
- */
-void initialize_ft_cache(void) {
- memset(ftc_num_msgs, 0, (65536 * sizeof(int)));
- memset(ftc_msgs, 0, (65536 * sizeof(long *)));
-}
-
-
-void ft_delete_remove(char *room, long msgnum)
-{
+void ft_delete_remove(char *room, long msgnum) {
if (room) return;
- /* Remove from fulltext index */
+ // Remove from fulltext index
if (CtdlGetConfigInt("c_enable_fulltext")) {
ft_index_message(msgnum, 0);
}
// Initialization function, called from modules_init.c
char *ctdl_module_init_fulltext(void) {
if (!threading) {
- initialize_ft_cache();
CtdlRegisterProtoHook(cmd_srch, "SRCH", "Full text search");
CtdlRegisterDeleteHook(ft_delete_remove);
- CtdlRegisterSearchFuncHook(ft_search, "fulltext");
- CtdlRegisterSessionHook(do_fulltext_indexing, EVT_TIMER, PRIO_CLEANUP + 300);
+ CtdlRegisterSessionHook(do_fulltext_indexing, EVT_HOUSE, PRIO_CLEANUP + 300);
}
- /* return our module name for the log */
+ // return our module name for the log
return "fulltext";
}