From 9aa7cafdfad0cb3a5639b247b81d1e7d96bc1572 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. --- 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 e54256f6bf2c0..d42b852423990 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 bf880250fe4d2..0770582e784dc 100644 --- a/src/v/cloud_storage/segment_chunk_data_source.cc +++ b/src/v/cloud_storage/segment_chunk_data_source.cc @@ -106,7 +106,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);