From 9a7a8fa86c6fd414030ede4b029cc444e34b535f Mon Sep 17 00:00:00 2001 From: Wojciech Mitros Date: Tue, 6 Jul 2021 19:55:48 +0200 Subject: [PATCH] mx sstable reader: remove false if case consume_row_marker_and_tombstone does not return proceed::no in the mp_row_consumer_m implementation, and even if it did, we would most likely want to yield proceed::no in that case as well. --- sstables/mx/reader.cc | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/sstables/mx/reader.cc b/sstables/mx/reader.cc index d4e065cf3cf2..43cf3897e0b7 100644 --- a/sstables/mx/reader.cc +++ b/sstables/mx/reader.cc @@ -575,11 +575,8 @@ class data_consume_rows_context_m : public data_consumer::continuous_data_consum } else { _state = state::ROW_BODY_MARKER; } - if (_consumer.consume_row_marker_and_tombstone( - _liveness, std::move(_row_tombstone), std::move(_row_shadowable_tombstone)) == consumer_m::proceed::no) { - _state = state::ROW_BODY_MISSING_COLUMNS; - co_yield consumer_m::proceed::yes; - } + _consumer.consume_row_marker_and_tombstone( + _liveness, std::move(_row_tombstone), std::move(_row_shadowable_tombstone)); } if (!_flags.has_all_columns()) { if (read_unsigned_vint(*_processing_data) != read_status::ready) {