commit 79a1bc5d5368bd780a2c0e745d28f562e248a59f
author Chen, Junjie <cjjnjust@gmail.com> Fri Aug 17 12:56:15 2018 -0400
committer Wes McKinney <wesm+git@apache.org> Fri Aug 17 12:56:15 2018 -0400
tree c624026fc8c5fea6b1183c2b2bb5d46d9a358ea0
parent ebb4b4b219916f57ac6a817734bbce13dae12d6c

PARQUET-1384: fix clang build error for bloom_filter-test.cc

replace https://github.com/apache/parquet-cpp/pull/488

Author: Chen, Junjie <cjjnjust@gmail.com>

Closes #490 from cjjnjust/parquet-1384 and squashes the following commits:

31afdd8 [Chen, Junjie] PARQUET-1384: fix clang build error for bloom_filter-test.cc

src/parquet/bloom_filter-test.cc

diff --git a/src/parquet/bloom_filter-test.cc b/src/parquet/bloom_filter-test.cc
index 69583af..96d2e06 100644
--- a/src/parquet/bloom_filter-test.cc
+++ b/src/parquet/bloom_filter-test.cc
@@ -165,7 +165,7 @@
 
   std::unique_ptr<uint8_t[]> bitset(new uint8_t[size]());
   std::shared_ptr<Buffer> buffer(new Buffer(bitset.get(), size));
-  handle->Read(size, &buffer);
+  PARQUET_THROW_NOT_OK(handle->Read(size, &buffer));
 
   InMemoryInputStream source(buffer);
   BlockSplitBloomFilter bloom_filter1 = BlockSplitBloomFilter::Deserialize(&source);
@@ -192,10 +192,10 @@
   bloom_filter2.WriteTo(&sink);
   std::shared_ptr<Buffer> buffer1 = sink.GetBuffer();
 
-  handle->Seek(0);
-  handle->GetSize(&size);
+  PARQUET_THROW_NOT_OK(handle->Seek(0));
+  PARQUET_THROW_NOT_OK(handle->GetSize(&size));
   std::shared_ptr<Buffer> buffer2;
-  handle->Read(size, &buffer2);
+  PARQUET_THROW_NOT_OK(handle->Read(size, &buffer2));
 
   EXPECT_TRUE((*buffer1).Equals(*buffer2));
 }