We've got +1's from Zoltan and Gabor. Ryan, you've committed a few BF patches that were written in response to your feedback on this list. Are you in a position to vote +1 now, or do you have further concerns we could address?
On 2019/07/31 02:17:15, 俊杰陈 <cjjnj...@gmail.com> wrote: > Dear Parquet developers > > We still need your vote! > > > On Wed, Jul 24, 2019 at 9:30 PM 俊杰陈 <cjjnj...@gmail.com> wrote: > > > > Hi @Ryan Blue @Wes McKinney > > > > We need your valuable vote, any feedback is welcome as well. > > > > On Tue, Jul 23, 2019 at 1:24 PM 俊杰陈 <cjjnj...@gmail.com> wrote: > > > > > > Call for voting again. > > > > > > On Fri, Jul 19, 2019 at 1:17 PM 俊杰陈 <cjjnj...@gmail.com> wrote: > > > > > > > > Dear Parquet developers > > > > > > > > We need more votes, please help to vote on this. > > > > > > > > On Wed, Jul 17, 2019 at 3:42 PM Gabor Szadovszky > > > > <gabor.szadovs...@cloudera.com.invalid> wrote: > > > > > > > > > > After getting in PARQUET-1625 I vote again for having bloom filter > > > > > spec and > > > > > the thrift file update as is in parquet-format master. > > > > > +1 (binding) > > > > > > > > > > On Mon, Jul 15, 2019 at 3:23 PM 俊杰陈 <cjjnj...@gmail.com> wrote: > > > > > > > > > > > Thanks Gabor, It's never too late to make it better. We don't have > > > > > > to > > > > > > run it in a hurry, it has been developed for a long time yet.:) > > > > > > > > > > > > The thrift file is indeed a bit lag behind the spec. As the spec > > > > > > defined, the bloom filter data is stored near the footer which means > > > > > > we don't have to handle it like the page. Therefore, I just opened a > > > > > > jira to remove bloom_filter_page_header in PageHeader structure, > > > > > > while > > > > > > the BloomFitlerHeader is kept intentionally for convenience. Since > > > > > > the > > > > > > spec and the thrift should be aligned with each other eventually, so > > > > > > the vote target is both of them. > > > > > > > > > > > > > > > > > > > > > > > > On Mon, Jul 15, 2019 at 7:48 PM Gabor Szadovszky > > > > > > <gabor.szadovs...@cloudera.com.invalid> wrote: > > > > > > > > > > > > > > Hi Junjie, > > > > > > > > > > > > > > Sorry for bringing up this a bit late but I have some problems > > > > > > > with the > > > > > > > format update. The parquet.thrift file is updated to have the > > > > > > > bloom > > > > > > filters > > > > > > > as a page (just as dictionaries and data pages). Meanwhile, the > > > > > > > spec > > > > > > > (BloomFilter.md) says that the bloom filter is stored near the > > > > > > > footer. > > > > > > So, > > > > > > > if the bloom filter is not part of the row-groups (like column > > > > > > > indexes) I > > > > > > > would not add it as a page. See the struct ColumnIndex in the > > > > > > > thrift > > > > > > file. > > > > > > > This struct is not referenced anywhere in it only declared. It > > > > > > > was done > > > > > > > this way because we don't parse it in the same way as we parse > > > > > > > the pages. > > > > > > > > > > > > > > Currently, I am not 100% sure about the target of this vote. If > > > > > > > it is a > > > > > > > vote about adding bloom filters in general then it is a +1 > > > > > > > (binding). If > > > > > > it > > > > > > > is about adding the bloom filters to parquet-format as is then, > > > > > > > it is a > > > > > > -1 > > > > > > > (binding) until we fix the issue above. > > > > > > > > > > > > > > Regards, > > > > > > > Gabor > > > > > > > > > > > > > > On Mon, Jul 15, 2019 at 11:45 AM Gidon Gershinsky > > > > > > > <gg5...@gmail.com> > > > > > > wrote: > > > > > > > > > > > > > > > +1 (non-binding) > > > > > > > > > > > > > > > > On Mon, Jul 15, 2019 at 12:08 PM Zoltan Ivanfi > > > > > > > > <z...@cloudera.com.invalid > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > +1 (binding) > > > > > > > > > > > > > > > > > > On Mon, Jul 15, 2019 at 9:57 AM 俊杰陈 <cjjnj...@gmail.com> > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > Dear Parquet developers > > > > > > > > > > > > > > > > > > > > I'd like to resume this vote, you can start to vote now. > > > > > > > > > > Thanks for > > > > > > > > your > > > > > > > > > time. > > > > > > > > > > > > > > > > > > > > On Wed, Jul 10, 2019 at 9:29 PM 俊杰陈 <cjjnj...@gmail.com> > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > I see, will resume this next week. Thanks. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Wed, Jul 10, 2019 at 5:26 PM Zoltan Ivanfi > > > > > > > > <z...@cloudera.com.invalid> > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > Hi Junjie, > > > > > > > > > > > > > > > > > > > > > > > > Since there are ongoing improvements addressing review > > > > > > comments, I > > > > > > > > > would > > > > > > > > > > > > hold off with the vote for a few more days until the > > > > > > specification > > > > > > > > > settles. > > > > > > > > > > > > > > > > > > > > > > > > Br, > > > > > > > > > > > > > > > > > > > > > > > > Zoltan > > > > > > > > > > > > > > > > > > > > > > > > On Wed, Jul 10, 2019 at 9:32 AM 俊杰陈 <cjjnj...@gmail.com> > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > Hi Parquet committers and developers > > > > > > > > > > > > > > > > > > > > > > > > > > We are waiting for your important ballot:) > > > > > > > > > > > > > > > > > > > > > > > > > > On Tue, Jul 9, 2019 at 10:21 AM 俊杰陈 > > > > > > > > > > > > > <cjjnj...@gmail.com> > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > Yes, there are some public benchmark results, such > > > > > > > > > > > > > > as the > > > > > > > > > official > > > > > > > > > > > > > > benchmark from xxhash site (http://www.xxhash.com/) > > > > > > > > > > > > > > and > > > > > > > > > published > > > > > > > > > > > > > > comparison from smhasher project > > > > > > > > > > > > > > (https://github.com/rurban/smhasher/). > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Tue, Jul 9, 2019 at 5:25 AM Wes McKinney < > > > > > > > > wesmck...@gmail.com> > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Do you have any benchmark data to support the > > > > > > > > > > > > > > > choice of > > > > > > hash > > > > > > > > > function? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Wed, Jul 3, 2019 at 8:41 AM 俊杰陈 > > > > > > > > > > > > > > > <cjjnj...@gmail.com> > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Dear Parquet developers > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > To simplify the voting, I 'd like to update > > > > > > > > > > > > > > > > voting > > > > > > content > > > > > > > > > to the > > > > > > > > > > > > > spec > > > > > > > > > > > > > > > > with xxHash hash strategy. Now you can reply > > > > > > > > > > > > > > > > with +1 > > > > > > or -1. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Thanks for your participation. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Tue, Jul 2, 2019 at 10:23 AM 俊杰陈 < > > > > > > cjjnj...@gmail.com> > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Dear Parquet developers > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Parquet Bloom filter has been developed for a > > > > > > > > > > > > > > > > > while, > > > > > > per > > > > > > > > > the > > > > > > > > > > > > > discussion on the mail list, it's time to call a vote > > > > > > > > > > > > > for > > > > > > spec to > > > > > > > > > move > > > > > > > > > > > > > forward. The current spec can be found at > > > > > > > > > > > > > > > > > > > > > > https://github.com/apache/parquet-format/blob/master/BloomFilter.md. > > > > > > > > > > > > > There are some different options about the internal > > > > > > > > > > > > > hash > > > > > > choice > > > > > > > > of > > > > > > > > > Bloom > > > > > > > > > > > > > filter and the PR is for that concern. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > So I 'd like to propose to vote the spec + > > > > > > > > > > > > > > > > > hash > > > > > > option, > > > > > > > > for > > > > > > > > > > > > > example: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > +1 to spec and xxHash > > > > > > > > > > > > > > > > > +1 to spec and murmur3 > > > > > > > > > > > > > > > > > ... > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Please help to vote, any feedback is also > > > > > > > > > > > > > > > > > welcome in > > > > > > the > > > > > > > > > > > > > discussion thread. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Thanks & Best Regards > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > > > > > > > > > Thanks & Best Regards > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > > > > > > > Thanks & Best Regards > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > > > > > > Thanks & Best Regards > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > > > > Thanks & Best Regards > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > > > Thanks & Best Regards > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > Thanks & Best Regards > > > > > > > > > > > > > > > > > > > > > > -- > > > > Thanks & Best Regards > > > > > > > > > > > > -- > > > Thanks & Best Regards > > > > > > > > -- > > Thanks & Best Regards > > > > -- > Thanks & Best Regards >