From 29f048816a31422685c44085ac20e45d97f95ecf Mon Sep 17 00:00:00 2001
From: Boris Kolpackov <boris@codesynthesis.com>
Date: Sat, 28 Sep 2013 10:51:12 +0200
Subject: Make schema version access (but not modification) thread-safe

Also cache the version in statements so that we don't have to lock
the mutex (slow) every time we need to check the version.
---
 odb/sqlite/section-statements.hxx | 13 +++++++++++++
 odb/sqlite/section-statements.txx |  1 +
 odb/sqlite/statements-base.hxx    | 24 +++++++++++++++++++-----
 3 files changed, 33 insertions(+), 5 deletions(-)

diff --git a/odb/sqlite/section-statements.hxx b/odb/sqlite/section-statements.hxx
index 8bdbda1..dff5ec5 100644
--- a/odb/sqlite/section-statements.hxx
+++ b/odb/sqlite/section-statements.hxx
@@ -10,12 +10,15 @@
 #include <cstddef> // std::size_t
 
 #include <odb/forward.hxx>
+#include <odb/schema-version.hxx>
 #include <odb/traits.hxx>
 
 #include <odb/sqlite/version.hxx>
 #include <odb/sqlite/sqlite-types.hxx>
 #include <odb/sqlite/binding.hxx>
 #include <odb/sqlite/statement.hxx>
+#include <odb/sqlite/connection.hxx>
+#include <odb/sqlite/database.hxx>
 #include <odb/sqlite/details/export.hxx>
 
 namespace odb
@@ -46,6 +49,15 @@ namespace odb
       connection_type&
       connection () {return conn_;}
 
+      const schema_version_migration&
+      version_migration (const char* name = "") const
+      {
+        if (svm_ == 0)
+          svm_ = &conn_.database ().schema_version_migration (name);
+
+        return *svm_;
+      }
+
       image_type&
       image () {return image_;}
 
@@ -138,6 +150,7 @@ namespace odb
 
     protected:
       connection_type& conn_;
+      mutable const schema_version_migration* svm_;
 
       // These come from object_statements.
       //
diff --git a/odb/sqlite/section-statements.txx b/odb/sqlite/section-statements.txx
index b6945d9..f097a5d 100644
--- a/odb/sqlite/section-statements.txx
+++ b/odb/sqlite/section-statements.txx
@@ -14,6 +14,7 @@ namespace odb
                         image_type& im,
                         binding& id, binding& idv)
         : conn_ (conn),
+          svm_ (0),
           image_ (im),
           id_binding_ (id),
           idv_binding_ (idv),
diff --git a/odb/sqlite/statements-base.hxx b/odb/sqlite/statements-base.hxx
index 5db1369..1df53cd 100644
--- a/odb/sqlite/statements-base.hxx
+++ b/odb/sqlite/statements-base.hxx
@@ -7,10 +7,12 @@
 
 #include <odb/pre.hxx>
 
+#include <odb/schema-version.hxx>
 #include <odb/details/shared-ptr.hxx>
 
 #include <odb/sqlite/version.hxx>
-#include <odb/sqlite/forward.hxx> // connection
+#include <odb/sqlite/connection.hxx>
+#include <odb/sqlite/database.hxx>
 
 #include <odb/sqlite/details/export.hxx>
 
@@ -29,18 +31,30 @@ namespace odb
         return conn_;
       }
 
+      // Schema version. database::schema_version_migration() is thread-
+      // safe which means it is also slow. Cache the result in statements
+      // so we can avoid the mutex lock. This is thread-safe since if the
+      // version is updated, then the statements cache will be expired.
+      //
+      const schema_version_migration&
+      version_migration (const char* name = "") const
+      {
+        if (svm_ == 0)
+          svm_ = &conn_.database ().schema_version_migration (name);
+
+        return *svm_;
+      }
+
     public:
       virtual
       ~statements_base ();
 
     protected:
-      statements_base (connection_type& conn)
-        : conn_ (conn)
-      {
-      }
+      statements_base (connection_type& conn): conn_ (conn), svm_ (0) {}
 
     protected:
       connection_type& conn_;
+      mutable const schema_version_migration* svm_;
     };
   }
 }
-- 
cgit v1.1