Skip to content

Commit

Permalink
[improve](move-memtable) reduce duplicate error report messages
Browse files Browse the repository at this point in the history
  • Loading branch information
kaijchen committed Jan 2, 2025
1 parent 77beb58 commit cf43ffc
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions be/src/runtime/load_stream.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,9 @@ Status TabletStream::init(std::shared_ptr<OlapTableSchemaParam> schema, int64_t

Status TabletStream::append_data(const PStreamHeader& header, butil::IOBuf* data) {
if (!_status.ok()) {
return _status.status();
// The error status is reported when it first occurs and
// doesn't need to be reported again until TabletStream::close().
return Status::OK();
}

// dispatch add_segment request
Expand Down Expand Up @@ -215,7 +217,9 @@ Status TabletStream::append_data(const PStreamHeader& header, butil::IOBuf* data

Status TabletStream::add_segment(const PStreamHeader& header, butil::IOBuf* data) {
if (!_status.ok()) {
return _status.status();
// The error status is reported when it first occurs and
// doesn't need to be reported again until TabletStream::close().
return Status::OK();
}

SCOPED_TIMER(_add_segment_timer);
Expand Down

0 comments on commit cf43ffc

Please sign in to comment.