This is an automated email from the ASF dual-hosted git repository.
yiguolei pushed a commit to branch branch-2.1
in repository https://gitbox.apache.org/repos/asf/doris.git
commit 9aa08d8deb15e34a1955b0496019b2ee698c8a0c
Author: zxealous <zhouchang...@baidu.com>
AuthorDate: Thu Apr 25 21:32:16 2024 +0800
[improve](disk) Not add disk path to broken list if check status is not
IO_ERROR (#34111)
---
be/src/olap/data_dir.cpp | 2 +-
be/src/service/doris_main.cpp | 1 +
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/be/src/olap/data_dir.cpp b/be/src/olap/data_dir.cpp
index 96b6c901295..72604bcca94 100644
--- a/be/src/olap/data_dir.cpp
+++ b/be/src/olap/data_dir.cpp
@@ -239,7 +239,7 @@ void DataDir::health_check() {
// check disk
if (_is_used) {
Status res = _read_and_write_test_file();
- if (!res) {
+ if (!res && res.is<IO_ERROR>()) {
LOG(WARNING) << "store read/write test file occur IO Error. path="
<< _path
<< ", err: " << res;
StorageEngine::instance()->add_broken_path(_path);
diff --git a/be/src/service/doris_main.cpp b/be/src/service/doris_main.cpp
index 60455300481..00aa4761467 100644
--- a/be/src/service/doris_main.cpp
+++ b/be/src/service/doris_main.cpp
@@ -435,6 +435,7 @@ int main(int argc, char** argv) {
it = paths.erase(it);
} else {
LOG(ERROR) << "read write test file failed, path=" << it->path;
+ // if only one disk and the disk is full, also need exit
because rocksdb will open failed
exit(-1);
}
} else {
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
