[#888,!573] use thread_local instead of ThreadResource

diff --git a/src/lib/pgsql/pgsql_connection.cc b/src/lib/pgsql/pgsql_connection.cc
index 38318843f98019f2d990789a35b27c0e8e2320e9..a2d7e6c56ed7c51cc736378981f1399446b1cf1c 100644 (file)
--- a/src/lib/pgsql/pgsql_connection.cc
+++ b/src/lib/pgsql/pgsql_connection.cc
@@ -188,6 +188,18 @@ PgSqlTransaction::commit() {
     committed_ = true;
 }
 
+PgSqlHolder&
+PgSqlConnection::handle() const {
+    thread_local std::shared_ptr<PgSqlHolder> result(std::make_shared<PgSqlHolder>());
+    if (connected_) {
+        result->openDatabase(*(const_cast<PgSqlConnection*>(this)));
+    }
+    if (prepared_) {
+        result->prepareStatements(*(const_cast<PgSqlConnection*>(this)));
+    }
+    return *result;
+}
+
 PgSqlConnection::~PgSqlConnection() {
     statements_.clear();
     handle().clear();

diff --git a/src/lib/pgsql/pgsql_connection.h b/src/lib/pgsql/pgsql_connection.h
index 54dd23cac118afcc79372c4aed211cf138c45738..c344f71d59a5810010290624b934edfc16764bb0 100644 (file)
--- a/src/lib/pgsql/pgsql_connection.h
+++ b/src/lib/pgsql/pgsql_connection.h
@@ -426,23 +426,11 @@ public:
     ///
     /// This field is public, because it is used heavily from PgSqlLeaseMgr
     /// and from PgSqlHostDataSource.
-    PgSqlHolder& handle() const {
-        auto result = handles_.resource();
-        if (connected_) {
-            result->openDatabase(*(const_cast<PgSqlConnection*>(this)));
-        }
-        if (prepared_) {
-            result->prepareStatements(*(const_cast<PgSqlConnection*>(this)));
-        }
-        return *result;
-    }
-
+    PgSqlHolder& handle() const;
 private:
     bool connected_;     ///< Flag to indicate openDatabase has been called
 
     bool prepared_;      ///< Flag to indicate prepareStatements has been called
-
-    mutable isc::dhcp::ThreadResource<PgSqlHolder> handles_;
 };
 
 }  // namespace db