Github user arina-ielchiieva commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1131#discussion_r170633434
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/expr/stat/ParquetPredicates.java
 ---
    @@ -66,6 +66,27 @@ public ParquetBooleanPredicate(String name, 
List<LogicalExpression> args, Expres
         }
       }
     
    +  public static abstract class ParquetIsPredicate extends 
LogicalExpressionBase implements ParquetFilterPredicate {
    --- End diff --
    
    `ParquetPredicates` class becomes too overloaded. Please split it into two: 
`ParquetComparisionPredicates` and `ParquetIsPredicate`.


---

Reply via email to