(druid) branch 31.0.0 updated: read metadata in SimpleQueryableIndex if available to compute segment ordering (#17181) (#17191)

[kfaraz]

This is an automated email from the ASF dual-hosted git repository.

kfaraz pushed a commit to branch 31.0.0
in repository https://gitbox.apache.org/repos/asf/druid.git


The following commit(s) were added to refs/heads/31.0.0 by this push:
     new 0ffdbaa6eb7 read metadata in SimpleQueryableIndex if available to 
compute segment ordering (#17181) (#17191)
0ffdbaa6eb7 is described below

commit 0ffdbaa6eb71abd0c3247687e8a0362ac4af3961
Author: Clint Wylie <cwy...@apache.org>
AuthorDate: Fri Oct 4 08:45:27 2024 -0700

    read metadata in SimpleQueryableIndex if available to compute segment 
ordering (#17181) (#17191)
---
 .../org/apache/druid/segment/SimpleQueryableIndex.java  | 17 ++++++++++-------
 1 file changed, 10 insertions(+), 7 deletions(-)

diff --git 
a/processing/src/main/java/org/apache/druid/segment/SimpleQueryableIndex.java 
b/processing/src/main/java/org/apache/druid/segment/SimpleQueryableIndex.java
index 3f6c0f5e825..9518876cb75 100644
--- 
a/processing/src/main/java/org/apache/druid/segment/SimpleQueryableIndex.java
+++ 
b/processing/src/main/java/org/apache/druid/segment/SimpleQueryableIndex.java
@@ -49,6 +49,7 @@ public abstract class SimpleQueryableIndex implements 
QueryableIndex
   private final Map<String, Supplier<ColumnHolder>> columns;
   private final SmooshedFileMapper fileMapper;
   private final Supplier<Map<String, DimensionHandler>> dimensionHandlers;
+  private final List<OrderBy> ordering;
 
   public SimpleQueryableIndex(
       Interval dataInterval,
@@ -83,6 +84,14 @@ public abstract class SimpleQueryableIndex implements 
QueryableIndex
     } else {
       this.dimensionHandlers = () -> 
initDimensionHandlers(availableDimensions);
     }
+
+    final Metadata metadata = getMetadata();
+    if (metadata != null && metadata.getOrdering() != null) {
+      this.ordering = metadata.getOrdering();
+    } else {
+      // When sort order isn't set in metadata.drd, assume the segment is 
sorted by __time.
+      this.ordering = Cursors.ascendingTimeOrder();
+    }
   }
 
   @Override
@@ -112,13 +121,7 @@ public abstract class SimpleQueryableIndex implements 
QueryableIndex
   @Override
   public List<OrderBy> getOrdering()
   {
-    final Metadata metadata = getMetadata();
-    if (metadata != null && metadata.getOrdering() != null) {
-      return metadata.getOrdering();
-    } else {
-      // When sort order isn't set in metadata.drd, assume the segment is 
sorted by __time.
-      return Cursors.ascendingTimeOrder();
-    }
+    return ordering;
   }
 
   @Override


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org
For additional commands, e-mail: commits-h...@druid.apache.org