From 5a4256338358c5853c1ef0f277c4884b4b7fbede Mon Sep 17 00:00:00 2001 From: Joel Rosdahl Date: Thu, 7 Jul 2022 12:32:26 +0200 Subject: [PATCH] chore: Improve Redis log and exception message --- src/storage/secondary/RedisStorage.cpp | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/src/storage/secondary/RedisStorage.cpp b/src/storage/secondary/RedisStorage.cpp index 4c98634e3..68bac59e4 100644 --- a/src/storage/secondary/RedisStorage.cpp +++ b/src/storage/secondary/RedisStorage.cpp @@ -114,10 +114,11 @@ RedisStorageBackend::RedisStorageBackend(const Params& params) ASSERT(url.scheme() == "redis" || url.scheme() == "redis+unix"); if (url.scheme() == "redis+unix" && !params.url.host().empty() && params.url.host() != "localhost") { - throw core::Fatal(FMT( - "invalid file path \"{}\": specifying a host (\"{}\") is not supported", - params.url.str(), - params.url.host())); + throw core::Fatal( + FMT("invalid file path \"{}\": specifying a host other than localhost is" + " not supported", + params.url.str(), + params.url.host())); } auto connect_timeout = k_default_connect_timeout; @@ -229,7 +230,7 @@ RedisStorageBackend::connect(const Url& url, const uint32_t operation_timeout) { if (url.scheme() == "redis+unix") { - LOG("Redis connecting to unix://{} (connect timeout {} ms)", + LOG("Redis connecting to {} (connect timeout {} ms)", url.path(), connect_timeout); m_context.reset(redisConnectUnixWithTimeout(url.path().c_str(), -- 2.47.3