somandal commented on code in PR #9333:
URL: https://github.com/apache/pinot/pull/9333#discussion_r974726393
##########
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/realtime/converter/stats/MutableColumnStatistics.java:
##########
@@ -145,6 +146,8 @@ public boolean isSorted() {
// Iterate over all data to figure out whether or not it's in sorted order
MutableForwardIndex mutableForwardIndex = (MutableForwardIndex)
_dataSource.getForwardIndex();
+ Preconditions.checkState(mutableForwardIndex != null,
Review Comment:
Jackie had a similar comment so I've gone ahead and unified it. I think the
exception message is enough to clarify when it's not supported due to a valid
disabled forward index and where the forward index shouldn't be null.
--
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]
