#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;
-
-int ftc_num_msgs[65536];
-long *ftc_msgs[65536];
+// 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;
// Compare function
}
-// Flush our index cache out to disk.
-void ft_flush_cache(void) {
- int i;
- time_t last_update = 0;
-
- cdb_begin_transaction();
- 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;
- }
- }
- cdb_end_transaction();
- 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)
void ft_index_message(long msgnum, int op) {
int i, j;
- Array *t = NULL;
+ 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);
- t = wordbreaker(txt);
+ tokens_in_this_message = wordbreaker(txt);
free(txt);
- syslog(LOG_DEBUG, "fulltext: %sindexing message %ld [%d tokens]", (op ? "" : "de"), msgnum, array_len(t));
- if (array_len(t) > 0) {
- for (i=0; i<array_len(t); ++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]
- memcpy(&tok, array_get_element_at(t, i), sizeof(int));
+ 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.ptr != NULL) {
- ftc_num_msgs[tok] = cdb_bucket.len / sizeof(long);
- ftc_msgs[tok] = malloc(cdb_bucket.len);
- memcpy(ftc_msgs[tok], cdb_bucket.ptr, cdb_bucket.len);
- }
- 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);
- }
- }
-
- array_free(t);
- }
-}
-
-// Add a message to the list of those to be indexed.
-void ft_index_msg(long msgnum, void *userdata) {
+ // Then write it back to disk
+ cdb_store(CDB_FULLTEXT, &tok, sizeof(int), newbucket, (nmsgs*sizeof(long)));
+ free(newbucket);
- 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)));
+ if (server_shutting_down) break;
}
- ft_newmsgs[ft_num_msgs - 1] = msgnum;
+ cdb_end_transaction();
+ end_critical_section(S_INDEXER);
+ CtdlSetConfigLong("MMfulltext", msgnum);
}
-
+ array_free(tokens_in_this_message);
}
// Scan a room for messages to index.
void ft_index_room(struct ctdlroom *qrbuf, void *data) {
- if (server_shutting_down)
+ if (server_shutting_down) {
return;
-
- CtdlGetRoom(&CC->room, qrbuf->QRname);
- CtdlForEachMessage(MSGS_ALL, 0L, NULL, NULL, NULL, ft_index_msg, NULL);
+ }
+
+ 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.
+
+ // 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);
+ }
}
int i;
static time_t last_progress = 0L;
static int is_running = 0;
+
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.
if (!CtdlGetConfigInt("c_enable_fulltext")) {
+ is_running = 0;
return;
}
-
// 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) {
// Silently return if our fulltext index is up to date with new messages.
if ((CtdlGetConfigLong("MMfulltext") >= CtdlGetConfigLong("MMhighest"))) {
+ 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);
-
- // 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;
- }
-
- // 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;
- }
+ messages_to_be_indexed = array_new(sizeof(long));
+ // 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
+
+ // 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) {
- Array *t = NULL;
- int i, j;
+Array *CtdlFullTextSearch(const char *search_string) {
+ int i, j, tok;
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;
+ long msgnum, smsgnum;
+ int count;
- t = wordbreaker(search_string);
- if (array_len(t) > 0) {
- for (i=0; i<array_len(t); ++i) {
+ Array *r = array_new(sizeof(long));
+ if (!r) return(NULL);
+ Array *t = wordbreaker(search_string);
- // search for tokens[i]
+ 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));
-
- // fetch the bucket, Liza
- if (ftc_msgs[tok] == NULL) {
- cdb_bucket = cdb_fetch(CDB_FULLTEXT, &tok, sizeof(int));
- if (cdb_bucket.ptr != NULL) {
- ftc_num_msgs[tok] = cdb_bucket.len / sizeof(long);
- ftc_msgs[tok] = (long *) malloc(cdb_bucket.len);
- memcpy(ftc_msgs[tok], cdb_bucket.ptr, cdb_bucket.len);
- }
- else {
- ftc_num_msgs[tok] = 0;
- ftc_msgs[tok] = malloc(sizeof(long));
+ 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;
}
-
}
- array_free(t);
- if (all_msgs != NULL) {
- qsort(all_msgs, num_all_msgs, sizeof(long), longcmp);
-
- // At this point, if a message appears array_len(t) times in the
- // list, then it contains all of the search tokens.
- if (num_all_msgs >= array_len(t))
- for (j=0; j<(num_all_msgs-array_len(t)+1); ++j) {
- if (all_msgs[j] == all_msgs[j+array_len(t)-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.
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;
}
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) {
if (room) return;
// 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 "fulltext";