You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by sa...@apache.org on 2016/03/17 02:01:30 UTC

[2/4] lucene-solr:branch_5_5: SOLR-8838: Returning non-stored docValues is incorrect for negative floats and doubles.

SOLR-8838: Returning non-stored docValues is incorrect for negative floats and doubles.


Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/44f9569d
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/44f9569d
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/44f9569d

Branch: refs/heads/branch_5_5
Commit: 44f9569d32a6b84126a91e39ddc598c374adeaab
Parents: 11fd447
Author: Steve Rowe <sa...@apache.org>
Authored: Wed Mar 16 18:56:23 2016 -0400
Committer: Steve Rowe <sa...@apache.org>
Committed: Wed Mar 16 19:54:12 2016 -0400

----------------------------------------------------------------------
 solr/CHANGES.txt                                                 | 2 ++
 solr/core/src/java/org/apache/solr/search/SolrIndexSearcher.java | 4 ++--
 2 files changed, 4 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/44f9569d/solr/CHANGES.txt
----------------------------------------------------------------------
diff --git a/solr/CHANGES.txt b/solr/CHANGES.txt
index 860ec2e..3c2f4ad 100644
--- a/solr/CHANGES.txt
+++ b/solr/CHANGES.txt
@@ -31,6 +31,8 @@ Bug Fixes
 * SOLR-8728: ReplicaAssigner throws NPE when a partial list of nodes are only participating in replica
   placement. splitshard should preassign nodes using rules, if rules are present (noble, Shai Erera)
 
+* SOLR-8838: Returning non-stored docValues is incorrect for negative floats and doubles.
+  (Ishan Chattopadhyaya, Steve Rowe)
 
 ======================= 5.5.0 =======================
 

http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/44f9569d/solr/core/src/java/org/apache/solr/search/SolrIndexSearcher.java
----------------------------------------------------------------------
diff --git a/solr/core/src/java/org/apache/solr/search/SolrIndexSearcher.java b/solr/core/src/java/org/apache/solr/search/SolrIndexSearcher.java
index 3990596..529aa33 100644
--- a/solr/core/src/java/org/apache/solr/search/SolrIndexSearcher.java
+++ b/solr/core/src/java/org/apache/solr/search/SolrIndexSearcher.java
@@ -821,9 +821,9 @@ public class SolrIndexSearcher extends IndexSearcher implements Closeable,SolrIn
               if (schemaField.getType() instanceof TrieIntField) {
                 val = ((Long) val).intValue();
               } else if (schemaField.getType() instanceof TrieFloatField) {
-                val = NumericUtils.sortableIntToFloat(((Long) val).intValue());
+                val = Float.intBitsToFloat(((Long)val).intValue());
               } else if (schemaField.getType() instanceof TrieDoubleField) {
-                val = NumericUtils.sortableLongToDouble((long) val);
+                val = Double.longBitsToDouble((long) val);
               } else if (schemaField.getType() instanceof TrieDateField) {
                 val = new Date((long) val);
               } else if (schemaField.getType() instanceof EnumField) {