summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorVolker Lendecke <vl@samba.org>2013-03-27 10:23:50 +0100
committerStefan Metzmacher <metze@samba.org>2013-03-27 13:40:15 +0100
commite8657887a9c1deadbd4e98c8b66cd46bdf36402a (patch)
tree977a12a753896831321b9fc2ed5558155cc857e0
parente3f5e474c826940ebaf43256cb06a50309ba3ef5 (diff)
downloadsamba-e8657887a9c1deadbd4e98c8b66cd46bdf36402a.tar.gz
samba-e8657887a9c1deadbd4e98c8b66cd46bdf36402a.tar.bz2
samba-e8657887a9c1deadbd4e98c8b66cd46bdf36402a.zip
dbwrap: Add a comment explaining a restriction
Signed-off-by: Volker Lendecke <vl@samba.org> Reviewed-by: Stefan Metzmacher <metze@samba.org> Autobuild-User(master): Stefan Metzmacher <metze@samba.org> Autobuild-Date(master): Wed Mar 27 13:40:15 CET 2013 on sn-devel-104
-rw-r--r--lib/dbwrap/dbwrap.c11
1 files changed, 11 insertions, 0 deletions
diff --git a/lib/dbwrap/dbwrap.c b/lib/dbwrap/dbwrap.c
index f0b7a9a3ac..f03514dc35 100644
--- a/lib/dbwrap/dbwrap.c
+++ b/lib/dbwrap/dbwrap.c
@@ -453,6 +453,17 @@ int dbwrap_get_seqnum(struct db_context *db)
int dbwrap_transaction_start(struct db_context *db)
{
if (!db->persistent) {
+ /*
+ * dbwrap_ctdb has two different data models for persistent
+ * and non-persistent databases. Transactions are supported
+ * only for the persistent databases. This check is here to
+ * prevent breakages of the cluster case, autobuild at this
+ * point only tests non-clustered Samba. Before removing this
+ * check, please make sure that this facility has also been
+ * added to dbwrap_ctdb.
+ *
+ * Thanks, vl
+ */
DEBUG(1, ("transactions not supported on non-persistent "
"database %s\n", db->name));
return -1;