rohityadav1993 commented on code in PR #16769:
URL: https://github.com/apache/pinot/pull/16769#discussion_r2331024800


##########
pinot-spi/src/main/java/org/apache/pinot/spi/data/readers/RecordReader.java:
##########
@@ -76,4 +76,15 @@ GenericRow next(GenericRow reuse)
    */
   void rewind()
       throws IOException;
+
+  /**
+   * Get the sorted document IDs if available.
+   * This method is used to retrieve the sorted order of document IDs for 
segment creation.
+   *
+   * @return array of sorted document IDs, or null if not available or not 
applicable
+   */
+  @Nullable
+  default int[] getSortedDocIds() {

Review Comment:
   Is this method needed in interface? We use this specifically in segment 
conversion and might not be applicable in general to recordReader interface.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to