facebook / facebook/rocksdb

Handle MultiRead return status

Open
#7,510 1 comment 0 reactions 0 assignees View on GitHub
up-for-grabs
Dominant language
C++
Stars
32.1k
Forks
6.9k
Avg merge
32m
Merged PRs (30d)
1

Description

> Note: Please use Issues only for bug reports. For questions, discussions, feature requests, etc. post to dev group: https://groups.google.com/forum/#!forum/rocksdb or https://www.facebook.com/groups/rocksdb.dev

### Expected behavior
In ```BlockBasedTable::RetrieveMultipleBlocks```, the return status of the ```MultiRead``` call should be handled - https://github.com/facebook/rocksdb/blob/master/table/block_based/block_based_table_reader.cc#L1696. In case of error, the error status should be reflected in each of the read requests.

### Actual behavior
The return status is ignored.

### Steps to reproduce the behavior
Read the code.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.