Default transactions strategies need to start a transaction even for …

…reads
plone · May 5, 2017 · 4a8a876 · 4a8a876
1 parent 5dc20d1
commit 4a8a876
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 5 deletions.
diff --git a/CHANGELOG.rst b/CHANGELOG.rst
@@ -1,7 +1,8 @@
 1.0.0a18 (unreleased)
 ---------------------
 
-- Nothing changed yet.
+- Default transactions strategies need to start a transaction even for reads
+  [vangheem]
 
 
 1.0.0a17 (2017-05-05)

diff --git a/config.json b/config.json
@@ -2,7 +2,7 @@
 	"databases": [{
 		"db": {
 			"storage": "postgresql",
-			"transaction_strategy": "none",
+			"transaction_strategy": "merge",
       "dsn": {
         "scheme": "postgres",
         "dbname": "guillotina",

diff --git a/guillotina/db/strategies/simple.py b/guillotina/db/strategies/simple.py
@@ -14,9 +14,6 @@ class SimpleStrategy(BaseStrategy):
     '''
 
     async def tpc_begin(self):
-        if not self.writable_transaction:
-            return
-
         await self._storage.start_transaction(self._transaction)
         tid = await self._storage.get_next_tid(self._transaction)
         if tid is not None: