Tomislav Novak 105c1e099b ForwardIterator::status() checks all child iterators
Summary:
Forward iterator only checked `status_` and `mutable_iter_->status()`, which is
not sufficient. For example, when reading exclusively from cache
(kBlockCacheTier), `mutable_iter_->status()` may return kOk (e.g. there's
nothing in the memtable), but one of immutable iterators could be in
kIncomplete. In this case, `ForwardIterator::status()` ought to return that
status instead of kOk.

This diff changes `status()` to also check `imm_iters_`, `l0_iters_`, and
`level_iters_`.

Test Plan:
  ROCKSDB_TESTS=TailingIteratorIncomplete ./db_test

Reviewers: ljin, igor

Reviewed By: igor

Subscribers: dhruba, leveldb

Differential Revision: https://reviews.facebook.net/D19581
2014-07-10 12:43:12 -07:00
..
2014-07-08 12:31:49 -07:00
2014-07-08 12:31:49 -07:00
2014-07-07 14:41:54 -07:00
2014-06-20 10:23:02 +02:00
2014-05-21 11:43:35 -07:00
2014-04-24 14:56:41 -04:00
2014-05-21 11:43:35 -07:00
2014-06-20 10:23:02 +02:00
2014-07-08 12:31:49 -07:00
2014-06-03 12:28:58 -07:00
2014-04-15 13:39:26 -07:00
2014-04-08 10:58:07 -07:00
2014-06-03 12:28:58 -07:00
2014-02-28 13:19:47 -08:00
2014-01-27 14:49:10 -08:00
2014-03-14 22:44:35 +00:00
2014-04-28 23:46:24 -04:00
2014-03-14 13:02:20 -07:00
2014-06-20 10:23:02 +02:00
2014-01-30 22:10:10 -08:00
2014-04-15 13:39:26 -07:00
2014-07-10 09:31:42 -07:00