Skip to content

PARQUET-173: Fixes StatisticsFilter for And filter predicate#108

Closed
liancheng wants to merge 2 commits intoapache:masterfrom
liancheng:PARQUET-173
Closed

PARQUET-173: Fixes StatisticsFilter for And filter predicate#108
liancheng wants to merge 2 commits intoapache:masterfrom
liancheng:PARQUET-173

Conversation

@liancheng
Copy link
Contributor

Review on Reviewable

@liancheng
Copy link
Contributor Author

@isnotinvain Would you mind to help review? Thanks.

@tianyi
Copy link

tianyi commented Jan 29, 2015

@tsdeng

@julienledem
Copy link
Member

+1 LGTM :)
@isnotinvain: what do you think?

@isnotinvain
Copy link
Contributor

+1, nice catch, had to think about it again to make sure but this looks correct :)

@asfgit asfgit closed this in 8041735 Feb 3, 2015
@liancheng liancheng deleted the PARQUET-173 branch February 5, 2015 01:34
rdblue pushed a commit to rdblue/parquet-mr that referenced this pull request Mar 9, 2015
<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/apache/incubator-parquet-mr/108)
<!-- Reviewable:end -->

Author: Cheng Lian <[email protected]>

Closes apache#108 from liancheng/PARQUET-173 and squashes the following commits:

d188f0b [Cheng Lian] Fixes test case
be2c8a1 [Cheng Lian] Fixes `StatisticsFilter` for `And` filter predicate
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants