shangxinli commented on code in PR #1023:
URL: https://github.com/apache/parquet-mr/pull/1023#discussion_r1174442027
##########
parquet-hadoop/src/main/java/org/apache/parquet/filter2/dictionarylevel/DictionaryFilter.java:
##########
@@ -52,14 +53,24 @@ public class DictionaryFilter implements
FilterPredicate.Visitor<Boolean> {
private static final boolean BLOCK_MIGHT_MATCH = false;
private static final boolean BLOCK_CANNOT_MATCH = true;
- public static boolean canDrop(FilterPredicate pred,
List<ColumnChunkMetaData> columns, DictionaryPageReadStore dictionaries) {
+ public static boolean canDrop(FilterPredicate pred,
List<ColumnChunkMetaData> columns,
+ DictionaryPageReadStore dictionaries, AtomicBoolean canExactlyDetermine) {
Review Comment:
This is a public method. We are changing the signature now. Can you add
another one to support callers that are still calling the old one?
--
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]
