diff options
| author | Eric Wong <e@80x24.org> | 2016-09-09 00:01:28 +0000 |
|---|---|---|
| committer | Eric Wong <e@80x24.org> | 2016-09-09 00:02:24 +0000 |
| commit | 4d9ad68e6565e488c9fa8e8e314178624827785c (patch) | |
| tree | bf885c388cbc95f525ed3f0b5345e71bba8ed40d /lib/PublicInbox/Search.pm | |
| parent | 1b14910344c10f85ce29281a5a7803ab3b2971be (diff) | |
| download | public-inbox-4d9ad68e6565e488c9fa8e8e314178624827785c.tar.gz | |
Specifying the "d:" field only worked for NumberValueRangeProcessor in older versions of Xapian, such as the one in Debian wheezy (libsearch-xapian-perl=1.2.10.0-1) This slipped through since I rarely use wheezy, anymore, and perhaps nobody else does, either. Perhaps wheezy support may be dropped, soon. Unfortunately, this requires a schema version bump.
Diffstat (limited to 'lib/PublicInbox/Search.pm')
| -rw-r--r-- | lib/PublicInbox/Search.pm | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/lib/PublicInbox/Search.pm b/lib/PublicInbox/Search.pm index c8e297f4..ceee39af 100644 --- a/lib/PublicInbox/Search.pm +++ b/lib/PublicInbox/Search.pm @@ -38,7 +38,8 @@ use constant { # 9 - disable Message-ID compression (SHA-1) # 10 - optimize doc for NNTP overviews # 11 - merge threads when vivifying ghosts - SCHEMA_VERSION => 11, + # 12 - change YYYYMMDD value column to numeric + SCHEMA_VERSION => 12, # n.b. FLAG_PURE_NOT is expensive not suitable for a public website # as it could become a denial-of-service vector @@ -221,7 +222,7 @@ sub qp { $qp->set_stemmer($self->stemmer); $qp->set_stemming_strategy(STEM_SOME); $qp->add_valuerangeprocessor( - Search::Xapian::StringValueRangeProcessor->new(YYYYMMDD, 'd:')); + Search::Xapian::NumberValueRangeProcessor->new(YYYYMMDD, 'd:')); while (my ($name, $prefix) = each %bool_pfx_external) { $qp->add_boolean_prefix($name, $prefix); |