Jackie-Jiang commented on a change in pull request #7654:
URL: https://github.com/apache/pinot/pull/7654#discussion_r738937559
##########
File path:
pinot-core/src/main/java/org/apache/pinot/core/query/pruner/ColumnValueSegmentPruner.java
##########
@@ -164,7 +274,73 @@ private boolean pruneEqPredicate(IndexSegment segment,
EqPredicate eqPredicate,
}
/**
- * For RANGE predicate, prune the segment based on:
+ * For IN predicate, prune the segments based on:
+ * <ul>
+ * <li>Column min/max value</li>
+ * <li>Column bloom filter</li>
+ * </ul>
+ * <p>NOTE: segments will not be pruned if the number of values is greater
than the threshold.
+ */
+ private boolean pruneInPredicate(IndexSegment segment, InPredicate
inPredicate,
Review comment:
I moved some methods around to keep the similar methods together, which
is identified as delete and add by github
--
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]
