Merge 0c89ba8 into 310ef06

apache · Aug 17, 2018 · 90c37de · 90c37de
2 parents 310ef06 + 0c89ba8
commit 90c37de
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/src/parquet/bloom_filter-test.cc b/src/parquet/bloom_filter-test.cc
@@ -72,7 +72,7 @@ TEST(BasicTest, TestBloomFilter) {
   // Deserialize Bloom filter from memory
   InMemoryInputStream source(sink.GetBuffer());
 
-  BlockSplitBloomFilter de_bloom = std::move(BlockSplitBloomFilter::Deserialize(&source));
+  BlockSplitBloomFilter de_bloom = BlockSplitBloomFilter::Deserialize(&source);
 
   for (int i = 0; i < 10; i++) {
     EXPECT_TRUE(de_bloom.FindHash(de_bloom.Hash(i)));
@@ -161,7 +161,7 @@ TEST(CompatibilityTest, TestBloomFilter) {
 
   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 =
@@ -189,10 +189,10 @@ TEST(CompatibilityTest, TestBloomFilter) {
   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));
 }