aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBoris Kolpackov <boris@codesynthesis.com>2013-09-28 10:51:12 +0200
committerBoris Kolpackov <boris@codesynthesis.com>2013-09-28 10:51:12 +0200
commit29f048816a31422685c44085ac20e45d97f95ecf (patch)
treedb49f030da3ee5c6598e03ee5f273381a838dc7b
parent0392ff4895fed7eb6a3f6befd53a07a7cd9990b4 (diff)
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.
-rw-r--r--odb/sqlite/section-statements.hxx13
-rw-r--r--odb/sqlite/section-statements.txx1
-rw-r--r--odb/sqlite/statements-base.hxx24
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_;
};
}
}