Ticket #2401 (closed defect: fixed)

Opened 3 years ago

Last modified 2 years ago

remove dead full-text indexing code

Reported by: glyph Assigned to: exarkun
Priority: normal Milestone:
Component: Mantissa Severity: normal
Keywords: Cc:
Estimated Completion (YYYY/MM/DD): Branch: branches/remove-xapwrap-2401
Author: exarkun

Description (Last modified by glyph)

I don't believe we ever intend to use xapian, or hype, again, but Mantissa's test suite still complains that their tests don't pass and we don't care.

Xapwrap is even listed on our retired projects page.

This code is all dead and rotting, and should be removed.

Change History

11/14/07 23:15:45 changed by glyph

  • description changed.

09/26/08 09:54:13 changed by exarkun

  • branch set to branches/remove-xapwrap-2401.
  • author set to exarkun.

(In [16734]) Branching to 'remove-xapwrap-2401'

09/26/08 10:29:06 changed by exarkun

  • keywords set to review.
  • owner changed from amir to glyph.

kazam.

10/01/08 15:23:32 changed by glyph

  • keywords deleted.
  • owner changed from glyph to exarkun.

Please add a docstring to XapianIndexer that indicates its status purely as a schema compatibility stub, and merge.

Given that there's still some dead code in here, perhaps there should be another ticket to totally remove it later. But I don't mind filing it later, if it becomes a problem, or next time I happen to be working on FTI code.

10/01/08 15:26:39 changed by exarkun

(In [16758]) Document the reason this apparently useless class exists

refs #2401

10/01/08 15:29:00 changed by exarkun

  • status changed from new to closed.
  • resolution set to fixed.

(In [16759]) Merge remove-xapwrap-2401

Author: exarkun Reviewer: glyph Fixes: #2401

Remove Xapwrap from the repository completely:

  1. It is unmaintained
  2. It does not work with the latest release of Xapian.

Remove the functionality in Mantissa which depended on Xapwrap, also.

jethro@divmod.org