From 695833082d28acd8d9d26812193ff1592f0f6977 Mon Sep 17 00:00:00 2001 From: dHannasch Date: Wed, 6 Jan 2021 12:00:58 -0700 Subject: [PATCH] [Redis] Note that each Redis Connect retry takes two minutes (#12183) * Slightly alter error message so it's the same in both cases. * Each retry takes about two minutes. --- src/ray/gcs/redis_context.cc | 14 +++----------- 1 file changed, 3 insertions(+), 11 deletions(-) diff --git a/src/ray/gcs/redis_context.cc b/src/ray/gcs/redis_context.cc index afc904d60..236b94bec 100644 --- a/src/ray/gcs/redis_context.cc +++ b/src/ray/gcs/redis_context.cc @@ -332,17 +332,9 @@ Status ConnectWithRetries(const std::string &address, int port, << errorMessage; break; } - if (*context == nullptr) { - RAY_LOG(WARNING) << errorMessage << " Will retry in " - << RayConfig::instance().redis_db_connect_wait_milliseconds() - << " milliseconds."; - } else if ((*context)->err) { - RAY_LOG(WARNING) << "Could not establish connection to Redis " << address << ":" - << port << " (context.err = " << (*context)->err - << "), will retry in " - << RayConfig::instance().redis_db_connect_wait_milliseconds() - << " milliseconds."; - } + RAY_LOG(WARNING) << errorMessage << " Will retry in " + << RayConfig::instance().redis_db_connect_wait_milliseconds() + << " milliseconds. Each retry takes about two minutes."; // Sleep for a little. std::this_thread::sleep_for(std::chrono::milliseconds( RayConfig::instance().redis_db_connect_wait_milliseconds()));