From e8657887a9c1deadbd4e98c8b66cd46bdf36402a Mon Sep 17 00:00:00 2001 From: Volker Lendecke Date: Wed, 27 Mar 2013 10:23:50 +0100 Subject: [PATCH] dbwrap: Add a comment explaining a restriction Signed-off-by: Volker Lendecke Reviewed-by: Stefan Metzmacher Autobuild-User(master): Stefan Metzmacher Autobuild-Date(master): Wed Mar 27 13:40:15 CET 2013 on sn-devel-104 --- lib/dbwrap/dbwrap.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/lib/dbwrap/dbwrap.c b/lib/dbwrap/dbwrap.c index f0b7a9a3ac5..f03514dc354 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; -- 2.34.1