honeymoose
/
OpenSearch
mirror of https://github.com/honeymoose/OpenSearch.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fixed positive infinity bug that can occur in specific scenarios when score mode average is used. 

Closes #4291
This commit is contained in:
Martijn van Groningen 2013-11-28 17:21:29 +01:00
parent baaa1a6aa2
commit 2ca5fd64fe
1 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/main/java/org/elasticsearch/index/search/child/ChildrenQuery.java
View File

@ -329,7 +329,6 @@ public class ChildrenQuery extends Query {
private final class AvgParentScorer extends ParentScorer {
HashedBytesArray currentUid;
final ObjectIntOpenHashMap<HashedBytesArray> uidToCount;
AvgParentScorer(Weight weight, IdReaderTypeCache idTypeCache, ObjectFloatOpenHashMap<HashedBytesArray> uidToScore, ObjectIntOpenHashMap<HashedBytesArray> uidToCount, DocIdSetIterator parentsIterator) {
@ -345,11 +344,11 @@ public class ChildrenQuery extends Query {
return currentDocId;
}
currentUid = idTypeCache.idByDoc(currentDocId);
currentScore = uidToScore.get(currentUid);
HashedBytesArray uid = idTypeCache.idByDoc(currentDocId);
currentScore = uidToScore.get(uid);
if (currentScore != 0) {
remaining--;
currentScore /= uidToCount.get(currentUid);
currentScore /= uidToCount.get(uid);
return currentDocId;
}
}
@ -366,7 +365,7 @@ public class ChildrenQuery extends Query {
currentScore = uidToScore.get(uid);
if (currentScore != 0) {
remaining--;
currentScore /= uidToCount.get(currentUid);
currentScore /= uidToCount.get(uid);
return currentDocId;
} else {
return nextDoc();