[LUCENE-1748] getPayloadSpans on org.apache.lucene.search.spans.SpanQuery should be abstract

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.4, 2.4.1
• Fix Version/s: 2.9
• Component/s: core/query/scoring

Environment

all

• Lucene Fields: New, Patch Available
• Assignee: Mark Miller
• Reporter: Hugh Cayless
• Created: 16/Jul/09 13:15
• Updated: 28/Aug/22 12:04
• Resolved: 12/Aug/09 20:33

Description

I just spent a long time tracking down a bug resulting from upgrading to Lucene 2.4.1 on a project that implements some SpanQuerys of its own and was written against 2.3. Since the project's SpanQuerys didn't implement getPayloadSpans, the call to that method went to SpanQuery.getPayloadSpans which returned null and caused a NullPointerException in the Lucene code, far away from the actual source of the problem.

It would be much better for this kind of thing to show up at compile time, I think.

Thanks!