diff --git a/be/src/cloud/cloud_tablet.cpp b/be/src/cloud/cloud_tablet.cpp index 7433b781c65d99..f54e0afee37df4 100644 --- a/be/src/cloud/cloud_tablet.cpp +++ b/be/src/cloud/cloud_tablet.cpp @@ -96,6 +96,7 @@ Status CloudTablet::capture_rs_readers(const Version& spec_version, auto missed_versions = get_missed_versions(spec_version.second); if (missed_versions.empty()) { st.set_code(VERSION_ALREADY_MERGED); // Reset error code + st.append(" versions are already compacted, "); } st.append(" tablet_id=" + std::to_string(tablet_id())); // clang-format off diff --git a/be/src/olap/tablet.cpp b/be/src/olap/tablet.cpp index 66278afdb666ee..177bda19b17362 100644 --- a/be/src/olap/tablet.cpp +++ b/be/src/olap/tablet.cpp @@ -855,8 +855,8 @@ Status Tablet::capture_consistent_versions_unlocked(const Version& spec_version, << ", version already has been merged. spec_version: " << spec_version << ", max_version: " << max_version_unlocked(); } - status = Status::Error( - "missed_versions is empty, spec_version " + status = Status::Error( + "versions are already compacted, spec_version " "{}, max_version {}, tablet_id {}", spec_version.second, max_version_unlocked(), tablet_id()); } else { diff --git a/be/src/olap/version_graph.cpp b/be/src/olap/version_graph.cpp index abdf82d8953ebf..010e7ce9fd89a5 100644 --- a/be/src/olap/version_graph.cpp +++ b/be/src/olap/version_graph.cpp @@ -566,8 +566,9 @@ void VersionGraph::_add_vertex_to_graph(int64_t vertex_value) { Status VersionGraph::capture_consistent_versions(const Version& spec_version, std::vector* version_path) const { if (spec_version.first > spec_version.second) { - return Status::Error("invalid specified version. spec_version={}-{}", - spec_version.first, spec_version.second); + return Status::Error( + "invalid specified version. spec_version={}-{}", spec_version.first, + spec_version.second); } int64_t cur_idx = -1; @@ -579,8 +580,9 @@ Status VersionGraph::capture_consistent_versions(const Version& spec_version, } if (cur_idx < 0) { - return Status::InternalError("failed to find path in version_graph. spec_version: {}-{}", - spec_version.first, spec_version.second); + return Status::InternalError( + "failed to find path in version_graph. spec_version: {}-{}", spec_version.first, + spec_version.second); } int64_t end_value = spec_version.second + 1; @@ -609,8 +611,9 @@ Status VersionGraph::capture_consistent_versions(const Version& spec_version, } cur_idx = next_idx; } else { - return Status::InternalError("fail to find path in version_graph. spec_version: {}-{}", - spec_version.first, spec_version.second); + return Status::InternalError( + "fail to find path in version_graph. spec_version: {}-{}", spec_version.first, + spec_version.second); } }