diff --git a/be/src/olap/tablet.cpp b/be/src/olap/tablet.cpp index 7c9a6ecfaa6fd5..22276e4041ea71 100644 --- a/be/src/olap/tablet.cpp +++ b/be/src/olap/tablet.cpp @@ -857,10 +857,14 @@ Status Tablet::capture_consistent_versions(const Version& spec_version, // if version_path is null, it may be a compaction check logic. // so to avoid print too many logs. if (version_path != nullptr) { - LOG(WARNING) << "tablet:" << full_name() - << ", version already has been merged. spec_version: " << spec_version; + LOG(WARNING) << "tablet:" << tablet_id() + << ", version already has been merged. spec_version: " << spec_version + << ", max_version: " << max_version_unlocked(); } - status = Status::Error("missed_versions is empty"); + status = Status::Error( + "missed_versions is empty, spec_version " + "{}, max_version {}, tablet_id {}", + spec_version.second, max_version_unlocked().second, tablet_id()); } else { if (version_path != nullptr) { LOG(WARNING) << "status:" << status << ", tablet:" << full_name() diff --git a/be/src/vec/exec/scan/new_olap_scanner.cpp b/be/src/vec/exec/scan/new_olap_scanner.cpp index 03c14ddafe03c5..42913697fcf9a3 100644 --- a/be/src/vec/exec/scan/new_olap_scanner.cpp +++ b/be/src/vec/exec/scan/new_olap_scanner.cpp @@ -189,9 +189,7 @@ Status NewOlapScanner::init() { auto st = _tablet->capture_rs_readers(rd_version, &read_source.rs_splits); if (!st.ok()) { LOG(WARNING) << "fail to init reader.res=" << st; - return Status::InternalError( - "failed to initialize storage reader. tablet_id={} : {}", - _tablet->tablet_id(), st.to_string()); + return st; } } if (!_state->skip_delete_predicate()) {