From 8c3bf0801bc7a6a841343ad93369d522a524c932 Mon Sep 17 00:00:00 2001 From: Nathan VanBenschoten Date: Fri, 25 May 2018 17:35:36 -0700 Subject: [PATCH] Check status when reading HashIndexPrefixesMetadataBlock Summary: This was missed in a refactor of `ReadBlockContents` (2f1a3a4). Closes https://github.com/facebook/rocksdb/pull/3906 Differential Revision: D8172648 Pulled By: ajkr fbshipit-source-id: 27e453b19795fea974bfed4721105be6f3a12090 --- table/block_based_table_reader.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/table/block_based_table_reader.cc b/table/block_based_table_reader.cc index c761ec082..e3783fe17 100644 --- a/table/block_based_table_reader.cc +++ b/table/block_based_table_reader.cc @@ -484,7 +484,7 @@ class HashIndexReader : public IndexReader { file, prefetch_buffer, footer, ReadOptions(), prefixes_meta_handle, &prefixes_meta_contents, ioptions, true /* decompress */, dummy_comp_dict /*compression dict*/, cache_options); - prefixes_meta_block_fetcher.ReadBlockContents(); + s = prefixes_meta_block_fetcher.ReadBlockContents(); if (!s.ok()) { // TODO: log error return Status::OK();