[#LUCENE-1303] BoostingTermQuery's explanation should be marked as Match even if the payload part negated or zero'ed it

Lucene - Java

BoostingTermQuery's explanation should be marked as Match even if the payload part negated or zero'ed it

Created: 10/Jun/08 04:33 PM Updated: 12/Jun/08 04:19 AM

Return to search

Component/s:

Query/Scoring

Affects Version/s:

None

Fix Version/s:

None

Time Tracking:

Not Specified

File Attachments:

	File Name	Date Attached	Attached By	Size
	lucene-1303.patch	2008-06-10 09:00 PM	Doron Cohen	2 kB

Lucene Fields:	New
Resolution Date:	12/Jun/08 04:19 AM

Description

« Hide

Since BTQ multiplies the payload on the score it might return a negative score.
The explanation should be marked as "Match" otherwise it is not added to container explanations,
See also in ~~LUCENE-1302~~.

All

Comments

Work Log

Change History

Subversion Commits

Sort Order:

Doron Cohen made changes - 10/Jun/08 04:34 PM

Field	Original Value	New Value
Description	Since BTQ multiplies the payload on the score it might return a negative score. The explanation should be marked as "Match" otherwise it is not added to container explanations, See also in Lucene-1302.	Since BTQ multiplies the payload on the score it might return a negative score. The explanation should be marked as "Match" otherwise it is not added to container explanations, See also in ~~LUCENE-1302~~.

[ Permlink | « Hide ]

Doron Cohen added a comment - 10/Jun/08 09:00 PM

Patch sets BTQ explanation as match according to score of non-payload part.

Also in this patch: while at it I noticed that TestBTQ sets indexing time boost to 0 and hence one assertion on search results was commented out.
I canceled that and re-enabled the assertion which now passes.

All core tests pass.

Doron Cohen made changes - 10/Jun/08 09:00 PM

Attachment

lucene-1303.patch [ 12383787 ]

Repository	Revision	Date	User	Message
ASF	#666949	Thu Jun 12 04:15:10 UTC 2008	doronc	~~LUCENE-1303~~: BoostingTermQuery's explanation marked as a Match depending only upon the non-payload part of the score.
				Files Changed
				MODIFY /lucene/java/trunk/src/java/org/apache/lucene/search/payloads/BoostingTermQuery.java MODIFY /lucene/java/trunk/src/test/org/apache/lucene/search/payloads/TestBoostingTermQuery.java MODIFY /lucene/java/trunk/CHANGES.txt

Doron Cohen made changes - 12/Jun/08 04:19 AM

Status	Open [ 1 ]	Resolved [ 5 ]
Resolution		Fixed [ 1 ]