krishan1390 commented on code in PR #17183:
URL: https://github.com/apache/pinot/pull/17183#discussion_r2532843579


##########
pinot-spi/src/main/java/org/apache/pinot/spi/data/readers/ColumnReader.java:
##########
@@ -59,6 +166,110 @@ public interface ColumnReader extends Closeable, 
Serializable {
   Object next()
       throws IOException;
 
+  /**
+   * Check if the next value to be read is null.
+   */
+  boolean isNextNull() throws IOException;
+
+  /**
+   * Move the reader to skip the next value in the column.
+   * This is typically called if isNextNull() returns true to skip the null 
value before calling nextInt(), etc
+   *  which can't handle null values.

Review Comment:
   updated



-- 
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