Tianyi Wang has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8170 )

Change subject: IMPALA-5789: Add always_false flag in bloom filter
......................................................................


Patch Set 2:

> Patch Set 1:
>
> (2 comments)
>
> Will wait for the new PS but had a couple of high-level bits of feedback 
> first. It may be worth looking at Thomas' patches if you haven't already to 
> make sure your changes are compatible.

This is ready for review. Thomas's min-max filter hasn't been merged and 
https://gerrit.cloudera.org/c/8148/ does not conflict with this patch.


--
To view, visit http://gerrit.cloudera.org:8080/8170
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If680240a3cd4583fc97c3192177d86d9567c4f8d
Gerrit-Change-Number: 8170
Gerrit-PatchSet: 2
Gerrit-Owner: Tianyi Wang <[email protected]>
Gerrit-Reviewer: Tianyi Wang <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Comment-Date: Fri, 06 Oct 2017 23:33:38 +0000
Gerrit-HasComments: No

Reply via email to