From ef6fec8e342bfc06bfb19f33906b695e3b00b4d5 Mon Sep 17 00:00:00 2001 From: Abhijat Malviya Date: Fri, 12 Apr 2024 17:30:12 +0530 Subject: [PATCH] cst: Downgrade error logs to debug Recently added error logs do not take into account sleep aborted and other shutdown related errors. Since the logs print exception details which are already logged down the call chain, and the new logs are for debugging hang in chunk read path, the new logs are downgraded to debug level in this change. (cherry picked from commit 9aa7cafdfad0cb3a5639b247b81d1e7d96bc1572) --- src/v/cloud_storage/segment_chunk_api.cc | 2 +- src/v/cloud_storage/segment_chunk_data_source.cc | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/v/cloud_storage/segment_chunk_api.cc b/src/v/cloud_storage/segment_chunk_api.cc index 9b35af5d426b8..277521eafb714 100644 --- a/src/v/cloud_storage/segment_chunk_api.cc +++ b/src/v/cloud_storage/segment_chunk_api.cc @@ -220,7 +220,7 @@ ss::future segment_chunks::hydrate_chunk( } } catch (const std::exception& ex) { vlog( - _ctxlog.error, + _ctxlog.debug, "Failed to hydrate chunk start {}, error: {}", chunk_start, ex.what()); diff --git a/src/v/cloud_storage/segment_chunk_data_source.cc b/src/v/cloud_storage/segment_chunk_data_source.cc index 1be33ebf6bbb2..2265fbf1832e3 100644 --- a/src/v/cloud_storage/segment_chunk_data_source.cc +++ b/src/v/cloud_storage/segment_chunk_data_source.cc @@ -107,7 +107,7 @@ ss::future<> chunk_data_source_impl::load_stream_for_chunk( } catch (...) { eptr = std::current_exception(); vlog( - _ctxlog.error, + _ctxlog.debug, "Hydrating chunk {} failed with error {}", chunk_start, eptr);