Github user sansanichfb commented on a diff in the pull request:
    --- Diff: 
    @@ -205,55 +217,138 @@ private boolean isFiltered(List<HivePartition> 
             return testOneFilter(partitionFields, filter, inputData);
    -    /*
    -     * We are testing one filter against all the partition fields. The 
    -     * has the form "fieldA = valueA". The partitions have the form
    -     * 
partitionOne=valueOne/partitionTwo=ValueTwo/partitionThree=valueThree 1.
    -     * For a filter to match one of the partitions, lets say partitionA for
    -     * example, we need: fieldA = partittionOne and valueA = valueOne. If 
    -     * condition occurs, we return true. 2. If fieldA does not match any 
one of
    -     * the partition fields we also return true, it means we ignore this 
    -     * because it is not on a partition field. 3. If fieldA = 
partittionOne and
    -     * valueA != valueOne, then we return false.
    -     */
    -    private boolean testOneFilter(List<HivePartition> partitionFields,
    -                                  Object filter, InputData input) {
    -        // Let's look first at the filter
    -        FilterParser.BasicFilter bFilter = (FilterParser.BasicFilter) 
    +    private boolean testForUnsupportedOperators(List<Object> filterList) {
    --- End diff --
    Method name is a bit confusing. Could we rename it to something like 

If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at or file a JIRA ticket
with INFRA.

Reply via email to