Changeset 2011

Timestamp:

2001-02-19T15:02:00+13:00 (23 years ago)

Author:

sjboddie

Message:

Set mg's accumulator method back to 'list' as the recent change appeared
to introduce a new (and more serious) bug while fixing the old bug. For
now we'll just have to live with it the way it is.

File:

• trunk/gsdl/src/colservr/mgsearch.cpp (modified) (3 diffs)

trunk/gsdl/src/colservr/mgsearch.cpp

@@ -28 +28 @@
 /*
    $Log$
+   Revision 1.31  2001/02/19 02:02:00  sjboddie
+   Set mg's accumulator method back to 'list' as the recent change appeared
+   to introduce a new (and more serious) bug while fixing the old bug. For
+   now we'll just have to live with it the way it is.
+
    Revision 1.30  2001/02/15 22:58:11  kjm18
    added a comment
@@ -440 +445 @@
 and in a different order. Minimal time tests dont show any speed improvement
 of list over array (maybe because its broken??).  [02/2001, kjm18]
+
+... [sjboddie, also 02/2001] turns out that changing the accumulator_method
+introduced a more serious bug than it fixed (i.e. occasionally when doing a
+ranked search for a very common word you get no results at all). I've
+changed it back to list for now, one day we should play with other
+accumulator_methods but for now I don't have time and don't want to risk
+introducing bugs (better the devil you know ;)
 */
 void mgsearchclass::setsearchmode (const queryparamclass &queryparams)
@@ -445 +457 @@
   mgq_ask(".set expert true");
   mgq_ask(".set sorted_terms true");
-  mgq_ask(".set accumulator_method array");
+  mgq_ask(".set accumulator_method list");
   mgq_ask(".set max_accumulators 500000");
   mgq_ask(".set maxparas 500000");