db: Use isValid(int32_t timeout) instead of isValid(void)

LandSandBoat · Dec 20, 2024 · 182c637 · 182c637
1 parent 32ee8de
commit 182c637
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/src/common/database.cpp b/src/common/database.cpp
@@ -84,7 +84,7 @@ mutex_guarded<db::detail::State>& db::detail::getState()
     {
         // If we have a valid and connected connection: return it
         // TODO: Does this logic make ping_connection redundant?
-        return state.connection != nullptr && (state.connection->isValid() || state.connection->reconnect());
+        return state.connection != nullptr && (state.connection->isValid(/*seconds*/1) || state.connection->reconnect());
     }))
     {
         return state;
@@ -105,10 +105,9 @@ mutex_guarded<db::detail::State>& db::detail::getState()
             auto host   = settings::get<std::string>("network.SQL_HOST");
             auto port   = settings::get<uint16>("network.SQL_PORT");
             auto schema = settings::get<std::string>("network.SQL_DATABASE");
-            auto url    = fmt::format("tcp://{}:{}", host, port);
+            auto url    = fmt::format("tcp://{}:{}/{}", host, port, schema);
 
             state.connection = std::unique_ptr<sql::Connection>(driver->connect(url.c_str(), login.c_str(), passwd.c_str()));
-            state.connection->setSchema(schema.c_str());
         }
         catch (const std::exception& e)
         {