]> code.citadel.org Git - citadel.git/blobdiff - citadel/serv_fulltext.c
* The full text indexer now runs in its own dedicated thread instead of
[citadel.git] / citadel / serv_fulltext.c
index c6be2cfd3f68321f79572239a8fa7906919d4a54..61e2085bda2cebaaeb13f02aa3692f44de970b65 100644 (file)
@@ -73,33 +73,42 @@ int longcmp(const void *rec1, const void *rec2) {
  * Index or de-index a message.  (op == 1 to index, 0 to de-index)
  */
 void ft_index_message(long msgnum, int op) {
-       struct CtdlMessage *msg;
        int num_tokens = 0;
        int *tokens = NULL;
        int i, j;
        struct cdbdata *cdb_bucket;
        int num_msgs;
        long *msgs;
+       char *msgtext;
 
        lprintf(CTDL_DEBUG, "ft_index_message() %s msg %ld\n",
                (op ? "adding" : "removing") , msgnum
        );
 
-       msg = CtdlFetchMessage(msgnum, 1);
-       if (msg == NULL) return;
-
-       if (msg->cm_fields['M'] != NULL) {
-               wordbreaker(msg->cm_fields['M'], &num_tokens, &tokens);
-       }
-       CtdlFreeMessage(msg);
-
+       /* Output the message as text before indexing it, so we don't end up
+        * indexing a bunch of encoded base64, etc.
+        */
+       CC->redirect_buffer = malloc(SIZ);
+       CC->redirect_len = 0;
+       CC->redirect_alloc = SIZ;
+       CtdlOutputMsg(msgnum, MT_CITADEL, HEADERS_ALL, 0, 1);
+       msgtext = CC->redirect_buffer;
+       CC->redirect_buffer = NULL;
+       CC->redirect_len = 0;
+       CC->redirect_alloc = 0;
+       lprintf(CTDL_DEBUG, "Wordbreaking message %ld...\n", msgnum);
+       wordbreaker(msgtext, &num_tokens, &tokens);
+       free(msgtext);
+
+       lprintf(CTDL_DEBUG, "Indexing message %ld...\n", msgnum);
        if (num_tokens > 0) {
                for (i=0; i<num_tokens; ++i) {
 
                        /* Add the message to the relevant token bucket */
 
-                       /* FIXME lock the file */
-                       cdb_bucket = cdb_fetch(CDB_FULLTEXT, &tokens[i], sizeof(int));
+                       /* FIXME do we need to lock the file here? */
+                       cdb_bucket = cdb_fetch(CDB_FULLTEXT, &tokens[i],
+                                                               sizeof(int));
                        if (cdb_bucket == NULL) {
                                cdb_bucket = malloc(sizeof(struct cdbdata));
                                cdb_bucket->len = 0;
@@ -112,23 +121,21 @@ void ft_index_message(long msgnum, int op) {
 
                        if (op == 1) {  /* add to index */
                                ++num_msgs;
-                               cdb_bucket->ptr = realloc(cdb_bucket->ptr, num_msgs*sizeof(long) );
+                               cdb_bucket->ptr = realloc(cdb_bucket->ptr,
+                                                       num_msgs*sizeof(long));
                                msgs = (long *) cdb_bucket->ptr;
                                msgs[num_msgs - 1] = msgnum;
                        }
 
                        if (op == 0) {  /* remove from index */
-                               /* FIXME do this */
-                       }
-
-                       /* sort and purge dups */
-                       if (num_msgs > 1) {
-                               qsort(msgs, num_msgs, sizeof(long), longcmp);
-                               for (j=0; j<(num_msgs-1); ++j) {
-                                       if (msgs[j] == msgs[j+1]) {
-                                               memmove(&msgs[j], &msgs[j+1],
-                                                       ((num_msgs - j)*sizeof(long)));
-                                               --num_msgs;
+                               if (num_msgs >= 1) {
+                               msgs = (long *) cdb_bucket->ptr;
+                                       for (j=0; j<num_msgs; ++j) {
+                                               if (msgs[j] == msgnum) {
+                                                       memmove(&msgs[j], &msgs[j+1], ((num_msgs - j - 1)*sizeof(long)));
+                                                       --num_msgs;
+                                                       --j;
+                                               }
                                        }
                                }
                        }
@@ -138,7 +145,7 @@ void ft_index_message(long msgnum, int op) {
 
                        cdb_free(cdb_bucket);
 
-                       /* FIXME unlock the file */
+                       /* FIXME do we need to unlock the file here? */
                }
 
                free(tokens);
@@ -180,6 +187,14 @@ void ft_index_room(struct ctdlroom *qrbuf, void *data)
 void do_fulltext_indexing(void) {
        int i;
        static time_t last_index = 0L;
+       static time_t last_progress = 0L;
+
+       /*
+        * Don't do this if the site doesn't have it enabled.
+        */
+       if (!config.c_enable_fulltext) {
+               return;
+       }
 
        /*
         * Make sure we don't run the indexer too frequently.
@@ -193,18 +208,15 @@ void do_fulltext_indexing(void) {
         * Check to see whether the fulltext index is up to date; if there
         * are no messages to index, don't waste any more time trying.
         */
-       lprintf(CTDL_DEBUG, "CitControl.MMhighest  = %ld\n", CitControl.MMhighest);
-       lprintf(CTDL_DEBUG, "CitControl.MMfulltext = %ld\n", CitControl.MMfulltext);
        if (CitControl.MMfulltext >= CitControl.MMhighest) {
-               /* nothing to do! */
-               return;
+               return;         /* nothing to do! */
        }
 
        lprintf(CTDL_DEBUG, "do_fulltext_indexing() started\n");
        
        /*
         * If we've switched wordbreaker modules, burn the index and start
-        * over.  FIXME write this...
+        * over.
         */
        if (CitControl.fulltext_wordbreaker != FT_WORDBREAKER_ID) {
                lprintf(CTDL_INFO, "(re)initializing full text index\n");
@@ -224,14 +236,31 @@ void do_fulltext_indexing(void) {
                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)*sizeof(long)));
+                                       ((ft_num_msgs - i - 1)*sizeof(long)));
                                --ft_num_msgs;
+                               --i;
                        }
                }
 
                /* Here it is ... do each message! */
                for (i=0; i<ft_num_msgs; ++i) {
+                       if ((time(NULL) - last_progress) > 10) {
+                               lprintf(CTDL_DEBUG,
+                                       "Indexed %d of %d messages (%d%%)\n",
+                                               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 (time_to_die) {
+                               lprintf(CTDL_DEBUG, "Indexer quitting early\n");
+                               ft_newhighest = ft_newmsgs[i];
+                               break;
+                       }
+
                }
 
                free(ft_newmsgs);
@@ -250,6 +279,31 @@ void do_fulltext_indexing(void) {
        return;
 }
 
+/*
+ * Main loop for the indexer thread.
+ */
+void *indexer_thread(void *arg) {
+       struct CitContext indexerCC;
+
+       lprintf(CTDL_DEBUG, "indexer_thread() initializing\n");
+
+       memset(&indexerCC, 0, sizeof(struct CitContext));
+       indexerCC.internal_pgm = 1;
+       indexerCC.cs_pid = 0;
+       pthread_setspecific(MyConKey, (void *)&indexerCC );
+
+       cdb_allocate_tsd();
+
+       while (!time_to_die) {
+               do_fulltext_indexing();
+               sleep(1);
+       }
+
+       lprintf(CTDL_DEBUG, "indexer_thread() exiting\n");
+       pthread_exit(NULL);
+}
+
+
 
 /*
  * API call to perform searches.
@@ -274,14 +328,19 @@ void ft_search(int *fts_num_msgs, long **fts_msgs, char *search_string) {
                for (i=0; i<num_tokens; ++i) {
 
                        /* search for tokens[i] */
-                       cdb_bucket = cdb_fetch(CDB_FULLTEXT, &tokens[i], sizeof(int));
+                       cdb_bucket = cdb_fetch(CDB_FULLTEXT, &tokens[i],
+                                                               sizeof(int));
                        if (cdb_bucket != NULL) {
                                num_msgs = cdb_bucket->len / sizeof(long);
                                msgs = (long *)cdb_bucket->ptr;
 
                                num_all_msgs += num_msgs;
-                               all_msgs = realloc(all_msgs, num_all_msgs*sizeof(long) );
-                               memcpy(&all_msgs[num_all_msgs - num_msgs], msgs, num_msgs*sizeof(long) );
+                               if (num_all_msgs > 0) {
+                                       all_msgs = realloc(all_msgs,
+                                               num_all_msgs*sizeof(long) );
+                                       memcpy(&all_msgs[num_all_msgs-num_msgs],
+                                               msgs, num_msgs*sizeof(long) );
+                               }
 
                                cdb_free(cdb_bucket);
                        }
@@ -294,15 +353,17 @@ void ft_search(int *fts_num_msgs, long **fts_msgs, char *search_string) {
                 * 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 (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];
+                                       ret_msgs = realloc(ret_msgs,
+                                               (num_ret_alloc*sizeof(long)) );
                                }
+                               ret_msgs[num_ret_msgs - 1] = all_msgs[j];
 
                        }
                }
@@ -325,10 +386,17 @@ void cmd_srch(char *argbuf) {
        char search_string[256];
 
        if (CtdlAccessCheck(ac_logged_in)) return;
+
+       if (!config.c_enable_fulltext) {
+               cprintf("%d Full text index is not enabled on this server.\n",
+                       ERROR + CMD_NOT_SUPPORTED);
+       }
+
        extract_token(search_string, argbuf, 0, '|', sizeof search_string);
        ft_search(&num_msgs, &msgs, search_string);
 
-       cprintf("%d %d msgs match all search words:\n", LISTING_FOLLOWS, num_msgs);
+       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]);
@@ -343,7 +411,6 @@ void cmd_srch(char *argbuf) {
 
 char *serv_fulltext_init(void)
 {
-       CtdlRegisterSessionHook(do_fulltext_indexing, EVT_TIMER);
-        CtdlRegisterProtoHook(cmd_srch, "SRCH", "Full text search");
+       CtdlRegisterProtoHook(cmd_srch, "SRCH", "Full text search");
        return "$Id$";
 }