You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@parquet.apache.org by ga...@apache.org on 2019/07/17 07:38:20 UTC

[parquet-format] branch master updated: PARQUET-1625: Align Bloom filter definition in parquet thrift with its spec (#145)

This is an automated email from the ASF dual-hosted git repository.

gabor pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/parquet-format.git


The following commit(s) were added to refs/heads/master by this push:
     new 2123f07  PARQUET-1625: Align Bloom filter definition in parquet thrift with its spec (#145)
2123f07 is described below

commit 2123f07bff0ca599c8cf4bfb0e852219144b247b
Author: Chen, Junjie <ji...@tencent.com>
AuthorDate: Wed Jul 17 15:38:15 2019 +0800

    PARQUET-1625: Align Bloom filter definition in parquet thrift with its spec (#145)
---
 src/main/thrift/parquet.thrift | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/main/thrift/parquet.thrift b/src/main/thrift/parquet.thrift
index 231705f..5bf3b7f 100644
--- a/src/main/thrift/parquet.thrift
+++ b/src/main/thrift/parquet.thrift
@@ -482,7 +482,6 @@ enum PageType {
   INDEX_PAGE = 1;
   DICTIONARY_PAGE = 2;
   DATA_PAGE_V2 = 3;
-  BLOOM_FILTER_PAGE = 4;
 }
 
 /**
@@ -587,7 +586,7 @@ union BloomFilterHash {
   * Bloom filter header is stored at beginning of Bloom filter data of each column
   * and followed by its bitset.
   **/
-struct BloomFilterPageHeader {
+struct BloomFilterHeader {
   /** The size of bitset in bytes **/
   1: required i32 numBytes;
   /** The algorithm for setting bits. **/
@@ -637,7 +636,6 @@ struct PageHeader {
   6: optional IndexPageHeader index_page_header;
   7: optional DictionaryPageHeader dictionary_page_header;
   8: optional DataPageHeaderV2 data_page_header_v2;
-  9: optional BloomFilterPageHeader bloom_filter_page_header;
 }
 
 /**