Write-only transactions can be assumed valid

djspiewak · Nov 23, 2008 · b659c9b · b659c9b
1 parent 3b6eea3
commit b659c9b
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/main/scala/com/codecommit/stm/Transaction.scala b/src/main/scala/com/codecommit/stm/Transaction.scala
@@ -57,9 +57,13 @@ final class Transaction private[stm] (val rev: Int) extends Context {
       writes foreach { _.lock() }
       try {
         val f = { ref: Ref[Any] => ref.rev == version(ref) }
-        val back = if (writes.size != 0) {      // allow snapshot read-points
+
+        // read or write-only transactions should not retry
+        val back = if ((writes.size | reads.size) == 0) {
+          true
+        } else {
           reads.forall(f) && writes.forall(f)
-        } else true
+        }
 
         if (back) {
           for (ref <- writes) {