Ticket #199 (new defect)

Opened 14 months ago

Last modified 7 months ago

Tidy up matchspy.h for inclusion in a release

Reported by: richard Owned by: richard
Priority: normal Milestone: 1.1.0
Component: Library API Version: SVN trunk
Severity: normal Keywords:
Cc: Blocked By: #198
Operating System: All Blocking:

Description (last modified by richard) (diff)

matchspy.h currently contains a set of matchspy implementations, and some associated functions, which are not yet in any release. We need to go through and tidy these up before the next release, before committing to supporting these.

Current status:

StringListSerialiser? is a bit ad-hoc - maybe it would be better simply to make values capable of storing multiple values. (See bug #198).

TermCountMatchSpy? should probably be removed - it goes against the explicitly documented "Match spies and match deciders shouldn't access anything except for document values", and is thus liable to be misused, leading to a bad impression of Xapian's speed. Richard is doing some tests to see how performance compares with a value based implementation (but this would also be helped if it was easier to store multiple values in a document).

CategorySelectMatchSpy? possibly shouldn't be a match spy - it could simply be a class (or a function) which is passed the results of ValueCountMatchSpy::get_values(), and returns a score for the categorisation, together with another function which converts the output of get_values() to a set of ranges.

Change History

Changed 14 months ago by richard

  • blocking set to 154

Changed 14 months ago by richard

  • blockedby set to 198

Changed 14 months ago by olly

  • status changed from new to assigned

Changed 14 months ago by richard

  • blocking changed from 154 to 200
  • summary changed from Tidy up matchspy.h before release to Tidy up matchspy.h for next release

Olly's backed the matchspy stuff out of SVN HEAD for now, so we've got until the 1.0.4 release to tidy this up. Changing to block #200 instead of #154.

Changed 11 months ago by olly

  • blocking changed from 200 to 160

Moving to 1.1

Changed 11 months ago by olly

  • summary changed from Tidy up matchspy.h for next release to Tidy up matchspy.h for inclusion in a release

Reword summary

Changed 11 months ago by trac

  • platform set to All

Changed 7 months ago by richard

  • description modified (diff)
  • milestone set to 1.1

Changed 7 months ago by richard

  • blocking deleted

Changed 7 months ago by olly

  • owner changed from newbugs to richard
  • status changed from assigned to new
Note: See TracTickets for help on using tickets.