You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ds...@apache.org on 2015/04/13 15:49:52 UTC

svn commit: r1673187 - in /lucene/dev/branches/branch_5x: ./ solr/ solr/core/ solr/core/src/java/org/apache/solr/highlight/DefaultSolrHighlighter.java

Author: dsmiley
Date: Mon Apr 13 13:49:52 2015
New Revision: 1673187

URL: http://svn.apache.org/r1673187
Log:
SOLR-6692: Don't round highlight fragment scores for ordering purposes

Modified:
    lucene/dev/branches/branch_5x/   (props changed)
    lucene/dev/branches/branch_5x/solr/   (props changed)
    lucene/dev/branches/branch_5x/solr/core/   (props changed)
    lucene/dev/branches/branch_5x/solr/core/src/java/org/apache/solr/highlight/DefaultSolrHighlighter.java

Modified: lucene/dev/branches/branch_5x/solr/core/src/java/org/apache/solr/highlight/DefaultSolrHighlighter.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_5x/solr/core/src/java/org/apache/solr/highlight/DefaultSolrHighlighter.java?rev=1673187&r1=1673186&r2=1673187&view=diff
==============================================================================
--- lucene/dev/branches/branch_5x/solr/core/src/java/org/apache/solr/highlight/DefaultSolrHighlighter.java (original)
+++ lucene/dev/branches/branch_5x/solr/core/src/java/org/apache/solr/highlight/DefaultSolrHighlighter.java Mon Apr 13 13:49:52 2015
@@ -564,7 +564,7 @@ public class DefaultSolrHighlighter exte
         Collections.sort(frags, new Comparator<TextFragment>() {//TODO make TextFragment Comparable
           @Override
           public int compare(TextFragment arg0, TextFragment arg1) {
-            return Math.round(arg1.getScore() - arg0.getScore());
+            return Float.compare(arg1.getScore(), arg0.getScore());
           }
         });
       }