This is an automated email from the ASF dual-hosted git repository.
airborne12 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new 451c70888d8 [fix](storage) Fix typo 'occured' in user-facing error
messages (#62521)
451c70888d8 is described below
commit 451c70888d8ed55906b98826624bc9612b2d11a8
Author: Mukunda Rao Katta <[email protected]>
AuthorDate: Mon May 18 00:59:37 2026 -0700
[fix](storage) Fix typo 'occured' in user-facing error messages (#62521)
### What problem does this PR solve?
Nine error messages in the storage BE misspell **'occurred'** as
**'occured'**. Because these strings are returned in runtime `Status`
error messages, they are visible to end users hitting inverted-index
build/read errors and storage-engine init errors.
### Files changed
- `be/src/storage/index/inverted/inverted_index_reader.cpp` — 3
occurrences
- `be/src/storage/task/index_builder.cpp` — 5 occurrences
- `be/src/storage/storage_engine.cpp` — 1 occurrence
Each change is to the error-message string only; no behavior change.
Co-authored-by: MukundaKatta <[email protected]>
---
be/src/storage/index/inverted/inverted_index_reader.cpp | 6 +++---
be/src/storage/storage_engine.cpp | 2 +-
be/src/storage/task/index_builder.cpp | 12 ++++++------
3 files changed, 10 insertions(+), 10 deletions(-)
diff --git a/be/src/storage/index/inverted/inverted_index_reader.cpp
b/be/src/storage/index/inverted/inverted_index_reader.cpp
index ec11fb5b28c..8eae5aaab66 100644
--- a/be/src/storage/index/inverted/inverted_index_reader.cpp
+++ b/be/src/storage/index/inverted/inverted_index_reader.cpp
@@ -360,10 +360,10 @@ Status InvertedIndexReader::match_index_search(
query->search(*term_match_bitmap);
}
} catch (const CLuceneError& e) {
- return
Status::Error<ErrorCode::INVERTED_INDEX_CLUCENE_ERROR>("CLuceneError occured:
{}",
+ return
Status::Error<ErrorCode::INVERTED_INDEX_CLUCENE_ERROR>("CLuceneError occurred:
{}",
e.what());
} catch (const Exception& e) {
- return
Status::Error<ErrorCode::INVERTED_INDEX_CLUCENE_ERROR>("Exception occured: {}",
+ return
Status::Error<ErrorCode::INVERTED_INDEX_CLUCENE_ERROR>("Exception occurred: {}",
e.what());
}
return Status::OK();
@@ -480,7 +480,7 @@ Status FullTextIndexReader::query(const
IndexQueryContextPtr& context,
return Status::OK();
} catch (const CLuceneError& e) {
return Status::Error<ErrorCode::INVERTED_INDEX_CLUCENE_ERROR>(
- "CLuceneError occured, error msg: {}", e.what());
+ "CLuceneError occurred, error msg: {}", e.what());
}
}
diff --git a/be/src/storage/storage_engine.cpp
b/be/src/storage/storage_engine.cpp
index 1a15929111d..7d6083134bd 100644
--- a/be/src/storage/storage_engine.cpp
+++ b/be/src/storage/storage_engine.cpp
@@ -289,7 +289,7 @@ static Status load_data_dirs(const std::vector<DataDir*>&
data_dirs) {
auto st = data_dir->load();
if (!st.ok()) {
- LOG(WARNING) << "error occured when init load tables. res=" <<
st
+ LOG(WARNING) << "error occurred when init load tables. res="
<< st
<< ", data dir=" << data_dir->path();
std::lock_guard lock(result_mtx);
result = std::move(st);
diff --git a/be/src/storage/task/index_builder.cpp
b/be/src/storage/task/index_builder.cpp
index 291a2c31d12..71176c99ce8 100644
--- a/be/src/storage/task/index_builder.cpp
+++ b/be/src/storage/task/index_builder.cpp
@@ -513,7 +513,7 @@ Status
IndexBuilder::handle_single_rowset(RowsetMetaSharedPtr output_rowset_meta
})
} catch (const std::exception& e) {
return
Status::Error<ErrorCode::INVERTED_INDEX_CLUCENE_ERROR>(
- "CLuceneError occured: {}", e.what());
+ "CLuceneError occurred: {}", e.what());
}
if (inverted_index_builder) {
@@ -543,7 +543,7 @@ Status
IndexBuilder::handle_single_rowset(RowsetMetaSharedPtr output_rowset_meta
})
} catch (const std::exception& e) {
return
Status::Error<ErrorCode::INVERTED_INDEX_CLUCENE_ERROR>(
- "CLuceneError occured: {}", e.what());
+ "CLuceneError occurred: {}", e.what());
}
if (index_writer) {
@@ -626,7 +626,7 @@ Status
IndexBuilder::handle_single_rowset(RowsetMetaSharedPtr output_rowset_meta
})
} catch (const std::exception& e) {
return
Status::Error<ErrorCode::INVERTED_INDEX_CLUCENE_ERROR>(
- "CLuceneError occured: {}", e.what());
+ "CLuceneError occurred: {}", e.what());
}
}
@@ -739,7 +739,7 @@ Status IndexBuilder::_add_nullable(const std::string&
column_name,
_index_column_writers[index_writer_sign]->add_array_nulls(null_map, num_rows));
} catch (const std::exception& e) {
return Status::Error<ErrorCode::INVERTED_INDEX_CLUCENE_ERROR>(
- "CLuceneError occured: {}", e.what());
+ "CLuceneError occurred: {}", e.what());
}
return Status::OK();
@@ -772,7 +772,7 @@ Status IndexBuilder::_add_nullable(const std::string&
column_name,
{ _CLTHROWA(CL_ERR_IO, "debug point:
_add_nullable_throw_exception"); })
} while (offset < num_rows);
} catch (const std::exception& e) {
- return
Status::Error<ErrorCode::INVERTED_INDEX_CLUCENE_ERROR>("CLuceneError occured:
{}",
+ return
Status::Error<ErrorCode::INVERTED_INDEX_CLUCENE_ERROR>("CLuceneError occurred:
{}",
e.what());
}
@@ -805,7 +805,7 @@ Status IndexBuilder::_add_data(const std::string&
column_name,
DBUG_EXECUTE_IF("IndexBuilder::_add_data_throw_exception",
{ _CLTHROWA(CL_ERR_IO, "debug point:
_add_data_throw_exception"); })
} catch (const std::exception& e) {
- return
Status::Error<ErrorCode::INVERTED_INDEX_CLUCENE_ERROR>("CLuceneError occured:
{}",
+ return
Status::Error<ErrorCode::INVERTED_INDEX_CLUCENE_ERROR>("CLuceneError occurred:
{}",
e.what());
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
