fixes for byte

Signed-off-by: Bharathwaj G <bharath78910@gmail.com>
opensearch-project · Sep 4, 2024 · 87abc91 · 87abc91
1 parent 747c06e
commit 87abc91
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 12 deletions.
diff --git a/server/src/main/java/org/opensearch/index/mapper/NumberFieldMapper.java b/server/src/main/java/org/opensearch/index/mapper/NumberFieldMapper.java
@@ -653,9 +653,7 @@ public byte[] encodePoint(Number value) {
 
             @Override
             public double toDoubleValue(long value) {
-                byte[] bytes = new byte[8];
-                NumericUtils.longToSortableBytes(value, bytes, 0);
-                return NumericUtils.sortableLongToDouble(NumericUtils.sortableBytesToLong(bytes, 0));
+                return objectToDouble(value);
             }
 
             @Override

diff --git a/...rg/opensearch/index/codec/composite99/datacube/startree/StarTreeDocValuesFormatTests.java b/...rg/opensearch/index/codec/composite99/datacube/startree/StarTreeDocValuesFormatTests.java
@@ -188,15 +188,15 @@ public void testStarTreeDocValues() throws IOException {
                 StarTreeDocument[] starTreeDocuments = StarTreeTestUtils.getSegmentsStarTreeDocuments(
                     List.of(starTreeValues),
                     List.of(
-                        StarTreeNumericType.DOUBLE,
-                        StarTreeNumericType.LONG,
-                        StarTreeNumericType.DOUBLE,
-                        StarTreeNumericType.DOUBLE,
-                        StarTreeNumericType.DOUBLE,
-                        StarTreeNumericType.LONG,
-                        StarTreeNumericType.DOUBLE,
-                        StarTreeNumericType.DOUBLE,
-                        StarTreeNumericType.LONG
+                        NumberFieldMapper.NumberType.DOUBLE,
+                        NumberFieldMapper.NumberType.LONG,
+                        NumberFieldMapper.NumberType.DOUBLE,
+                        NumberFieldMapper.NumberType.DOUBLE,
+                        NumberFieldMapper.NumberType.DOUBLE,
+                        NumberFieldMapper.NumberType.LONG,
+                        NumberFieldMapper.NumberType.DOUBLE,
+                        NumberFieldMapper.NumberType.DOUBLE,
+                        NumberFieldMapper.NumberType.LONG
                     ),
                     reader.maxDoc()
                 );