batch

h2/src/main/org/h2/mvstore/Page.java

@@ -836,6 +836,8 @@ public abstract class Page implements Cloneable
         }
     }
 
+    public abstract CursorPos getAppendCursorPos(CursorPos cursorPos);
+
     public abstract void removeAllRecursive();
 
     private Object[] createKeyStorage(int size)
@@ -1083,6 +1085,13 @@ public abstract class Page implements Cloneable
             removePage();
         }
 
+        @Override
+        public CursorPos getAppendCursorPos(CursorPos cursorPos) {
+            int keyCount = getKeyCount();
+            Page childPage = getChildPage(keyCount);
+            return childPage.getAppendCursorPos(new CursorPos(this, keyCount, cursorPos));
+        }
+
         @Override
         protected void readPayLoad(ByteBuffer buff) {
             int keyCount = getKeyCount();
@@ -1322,6 +1331,12 @@ public abstract class Page implements Cloneable
             removePage();
         }
 
+        @Override
+        public CursorPos getAppendCursorPos(CursorPos cursorPos) {
+            int keyCount = getKeyCount();
+            return new CursorPos(this, -keyCount - 1, cursorPos);
+        }
+
         @Override
         protected void readPayLoad(ByteBuffer buff) {
             int keyCount = getKeyCount();

h2/src/main/org/h2/mvstore/tx/Transaction.java

@@ -300,7 +300,7 @@ public class Transaction {
         }
         int currentStatus = getStatus(currentState);
         checkOpen(currentStatus);
-        store.removeUndoLogRecord(transactionId, logId);
+        store.removeUndoLogRecord(transactionId);
     }
 
     /**

h2/src/main/org/h2/mvstore/tx/TransactionMap.java

@@ -88,9 +88,9 @@ public class TransactionMap<K, V> {
             for (int i = opentransactions.nextSetBit(0); i >= 0; i = opentransactions.nextSetBit(i+1)) {
                 MVMap<Long, Object[]> undoLog = store.undoLogs[i];
                 if (undoLog != null) {
-                    MVMap.RootReference rootReference = undoLog.getRoot();
+                    MVMap.RootReference rootReference = undoLog.flushAppendBuffer();
                     undoLogRootReferences[i] = rootReference;
-                    undoLogSize += rootReference.root.getTotalCount();
+                    undoLogSize += rootReference.root.getTotalCount() + rootReference.appendCounter;
                 }
             }
         } while(committingTransactions != store.committingTransactions.get() ||

h2/src/main/org/h2/mvstore/tx/TransactionStore.java

@@ -16,6 +16,7 @@ import org.h2.mvstore.Cursor;
 import org.h2.mvstore.DataUtils;
 import org.h2.mvstore.MVMap;
 import org.h2.mvstore.MVStore;
+import org.h2.mvstore.Page;
 import org.h2.mvstore.WriteBuffer;
 import org.h2.mvstore.type.DataType;
 import org.h2.mvstore.type.ObjectDataType;
@@ -132,7 +133,9 @@ public class TransactionStore {
         ArrayType undoLogValueType = new ArrayType(new DataType[]{
                 new ObjectDataType(), dataType, oldValueType
         });
-        undoLogBuilder = new MVMap.Builder<Long, Object[]>().valueType(undoLogValueType);
+        undoLogBuilder = new MVMap.Builder<Long, Object[]>()
+                .singleWriter()
+                .valueType(undoLogValueType);
     }
 
     /**
@@ -381,24 +384,16 @@ public class TransactionStore {
                     "is still open: {0}",
                     transactionId);
         }
-        undoLog.put(undoKey, undoLogRecord);
+        undoLog.append(undoKey, undoLogRecord);
         return undoKey;
     }
 
     /**
      * Remove an undo log entry.
      * @param transactionId id of the transaction
-     * @param logId sequential number of the log record within transaction
      */
-    public void removeUndoLogRecord(int transactionId, long logId) {
-        Long undoKey = getOperationId(transactionId, logId);
-        Object[] old = undoLogs[transactionId].remove(undoKey);
-        if (old == null) {
-            throw DataUtils.newIllegalStateException(
-                    DataUtils.ERROR_TRANSACTION_ILLEGAL_STATE,
-                    "Transaction {0} was concurrently rolled back",
-                    transactionId);
-        }
+    void removeUndoLogRecord(int transactionId) {
+        undoLogs[transactionId].trimLast();
     }
 
     /**
@@ -432,7 +427,9 @@ public class TransactionStore {
                     store.renameMap(undoLog, getUndoLogName(true, transactionId));
                 }
                 try {
-                    Cursor<Long, Object[]> cursor = undoLog.cursor(null);
+                    MVMap.RootReference rootReference = undoLog.flushAppendBuffer();
+                    Page rootPage = rootReference.root;
+                    Cursor<Long, Object[]> cursor = new Cursor<>(rootPage, null);
                     while (cursor.hasNext()) {
                         Long undoKey = cursor.next();
                         Object[] op = cursor.getValue();
@@ -587,6 +584,7 @@ public class TransactionStore {
     void rollbackTo(Transaction t, long maxLogId, long toLogId) {
         int transactionId = t.getId();
         MVMap<Long, Object[]> undoLog = undoLogs[transactionId];
+        undoLog.flushAppendBuffer();
         RollbackDecisionMaker decisionMaker = new RollbackDecisionMaker(this, transactionId, toLogId, t.listener);
         for (long logId = maxLogId - 1; logId >= toLogId; logId--) {
             Long undoKey = getOperationId(transactionId, logId);
@@ -608,6 +606,7 @@ public class TransactionStore {
             final long toLogId) {
 
         final MVMap<Long, Object[]> undoLog = undoLogs[t.getId()];
+        undoLog.flushAppendBuffer();
         return new Iterator<Change>() {
 
             private long logId = maxLogId - 1;