edits
[iotcloud.git] / version2 / src / C / Table.cc
index 76a7f16adfc7b6a7431859fa3742ccf1f31db7dd..d70a590890b170fac9417fc6ab049437304de286 100644 (file)
@@ -1,6 +1,38 @@
 #include "Table.h"
+#include "CloudComm.h"
+#include "SlotBuffer.h"
+#include "NewKey.h"
+#include "Slot.h"
+#include "KeyValue.h"
+#include "Error.h"
+#include "PendingTransaction.h"
+#include "TableStatus.h"
+#include "TransactionStatus.h"
+#include "Transaction.h"
+#include "LastMessage.h"
+#include "SecureRandom.h"
+#include "ByteBuffer.h"
+#include "Abort.h"
+#include "CommitPart.h"
+#include "ArbitrationRound.h"
+#include "TransactionPart.h"
+#include "Commit.h"
+#include "RejectedMessage.h"
+#include "SlotIndexer.h"
+#include <stdlib.h>
+
+int compareInt64(const void *a, const void *b) {
+       const int64_t *pa = (const int64_t *) a;
+       const int64_t *pb = (const int64_t *) b;
+       if (*pa < *pb)
+               return -1;
+       else if (*pa > *pb)
+               return 1;
+       else
+               return 0;
+}
 
-Table::Table(String baseurl, String password, int64_t _localMachineId, int listeningPort) :
+Table::Table(IoTString *baseurl, IoTString *password, int64_t _localMachineId, int listeningPort) :
        buffer(NULL),
        cloud(new CloudComm(this, baseurl, password, listeningPort)),
        random(NULL),
@@ -11,7 +43,7 @@ Table::Table(String baseurl, String password, int64_t _localMachineId, int liste
        numberOfSlots(0),
        bufferResizeThreshold(0),
        liveSlotCount(0),
-       oldestLiveSlotSequenceNumber(0),
+       oldestLiveSlotSequenceNumver(1),
        localMachineId(_localMachineId),
        sequenceNumber(0),
        localTransactionSequenceNumber(0),
@@ -20,7 +52,7 @@ Table::Table(String baseurl, String password, int64_t _localMachineId, int liste
        localArbitrationSequenceNumber(0),
        hadPartialSendToServer(false),
        attemptedToSendToServer(false),
-       expectedSize(0),
+       expectedsize(0),
        didFindTableStatus(false),
        currMaxSize(0),
        lastSlotAttemptedToSend(NULL),
@@ -62,7 +94,7 @@ Table::Table(String baseurl, String password, int64_t _localMachineId, int liste
        init();
 }
 
-Table::Table(CloudComm _cloud, int64_t _localMachineId) :
+Table::Table(CloudComm *_cloud, int64_t _localMachineId) :
        buffer(NULL),
        cloud(_cloud),
        random(NULL),
@@ -73,7 +105,7 @@ Table::Table(CloudComm _cloud, int64_t _localMachineId) :
        numberOfSlots(0),
        bufferResizeThreshold(0),
        liveSlotCount(0),
-       oldestLiveSlotSequenceNumber(0),
+       oldestLiveSlotSequenceNumver(1),
        localMachineId(_localMachineId),
        sequenceNumber(0),
        localTransactionSequenceNumber(0),
@@ -82,7 +114,7 @@ Table::Table(CloudComm _cloud, int64_t _localMachineId) :
        localArbitrationSequenceNumber(0),
        hadPartialSendToServer(false),
        attemptedToSendToServer(false),
-       expectedSize(0),
+       expectedsize(0),
        didFindTableStatus(false),
        currMaxSize(0),
        lastSlotAttemptedToSend(NULL),
@@ -128,148 +160,64 @@ Table::Table(CloudComm _cloud, int64_t _localMachineId) :
  * Init all the stuff needed for for table usage
  */
 void Table::init() {
-
        // Init helper objects
-       random = new Random();
+       random = new SecureRandom();
        buffer = new SlotBuffer();
 
-       // Set Variables
-       oldestLiveSlotSequenceNumver = 1;
-
        // init data structs
-       committedKeyValueTable = new Hashtable<IoTString, KeyValue>();
-       speculatedKeyValueTable = new Hashtable<IoTString, KeyValue>();
-       pendingTransactionSpeculatedKeyValueTable = new Hashtable<IoTString, KeyValue>();
-       liveNewKeyTable = new Hashtable<IoTString, NewKey>();
-       lastMessageTable = new Hashtable<int64_t Pair<int64_t Liveness> >();
-       rejectedMessageWatchVectorTable = new Hashtable<int64_t Hashset<RejectedMessage> >();
-       arbitratorTable = new Hashtable<IoTString, Long>();
-       liveAbortTable = new Hashtable<Pair<int64_t, int64_t>, Abort>();
-       newTransactionParts = new Hashtable<int64_t Hashtable<Pair<int64_t int32_t>, TransactionPart> >();
-       newCommitParts = new Hashtable<int64_t Hashtable<Pair<int64_t int32_t>, CommitPart> >();
+       committedKeyValueTable = new Hashtable<IoTString *, KeyValue *>();
+       speculatedKeyValueTable = new Hashtable<IoTString *, KeyValue *>();
+       pendingTransactionSpeculatedKeyValueTable = new Hashtable<IoTString *, KeyValue *>();
+       liveNewKeyTable = new Hashtable<IoTString *, NewKey *>();
+       lastMessageTable = new Hashtable<int64_t, Pair<int64_t, Liveness *> * >();
+       rejectedMessageWatchVectorTable = new Hashtable<int64_t, Hashset<RejectedMessage *> * >();
+       arbitratorTable = new Hashtable<IoTString *, int64_t>();
+       liveAbortTable = new Hashtable<Pair<int64_t, int64_t> *, Abort *, uintptr_t, 0, pairHashFunction, pairEquals>();
+       newTransactionParts = new Hashtable<int64_t, Hashtable<Pair<int64_t, int32_t> *, TransactionPart *, uintptr_t, 0, pairHashFunction, pairEquals> *>();
+       newCommitParts = new Hashtable<int64_t, Hashtable<Pair<int64_t, int32_t> *, CommitPart *, uintptr_t, 0, pairHashFunction, pairEquals> *>();
        lastArbitratedTransactionNumberByArbitratorTable = new Hashtable<int64_t, int64_t>();
-       liveTransactionBySequenceNumberTable = new Hashtable<int64_t Transaction>();
-       liveTransactionByTransactionIdTable = new Hashtable<Pair<int64_t, int64_t>, Transaction>();
-       liveCommitsTable = new Hashtable<int64_t Hashtable<int64_t Commit> >();
-       liveCommitsByKeyTable = new Hashtable<IoTString, Commit>();
+       liveTransactionBySequenceNumberTable = new Hashtable<int64_t, Transaction *>();
+       liveTransactionByTransactionIdTable = new Hashtable<Pair<int64_t, int64_t> *, Transaction *, uintptr_t, 0, pairHashFunction, pairEquals>();
+       liveCommitsTable = new Hashtable<int64_t, Hashtable<int64_t, Commit *> * >();
+       liveCommitsByKeyTable = new Hashtable<IoTString *, Commit *>();
        lastCommitSeenSequenceNumberByArbitratorTable = new Hashtable<int64_t, int64_t>();
-       rejectedSlotVector = new Vector<Long>();
-       pendingTransactionQueue = new Vector<Transaction>();
-       pendingSendArbitrationEntriesToDelete = new Vector<Entry>();
-       transactionPartsSent = new Hashtable<Transaction, Vector<int32_t> >();
-       outstandingTransactionStatus = new Hashtable<int64_t TransactionStatus>();
-       liveAbortsGeneratedByLocal = new Hashtable<int64_t Abort>();
-       offlineTransactionsCommittedAndAtServer = new Hashset<Pair<int64_t, int64_t> >();
-       localCommunicationTable = new Hashtable<int64_t Pair<String, int32_t> >();
+       rejectedSlotVector = new Vector<int64_t>();
+       pendingTransactionQueue = new Vector<Transaction *>();
+       pendingSendArbitrationEntriesToDelete = new Vector<Entry *>();
+       transactionPartsSent = new Hashtable<Transaction *, Vector<int32_t> *>();
+       outstandingTransactionStatus = new Hashtable<int64_t, TransactionStatus *>();
+       liveAbortsGeneratedByLocal = new Hashtable<int64_t, Abort *>();
+       offlineTransactionsCommittedAndAtServer = new Hashset<Pair<int64_t, int64_t> *, uintptr_t, 0, pairHashFunction, pairEquals>();
+       localCommunicationTable = new Hashtable<int64_t, Pair<IoTString *, int32_t> *>();
        lastTransactionSeenFromMachineFromServer = new Hashtable<int64_t, int64_t>();
-       pendingSendArbitrationRounds = new Vector<ArbitrationRound>();
+       pendingSendArbitrationRounds = new Vector<ArbitrationRound *>();
        lastArbitrationDataLocalSequenceNumberSeenFromArbitrator = new Hashtable<int64_t, int64_t>();
 
-
        // Other init stuff
-       numberOfSlots = buffer.capacity();
+       numberOfSlots = buffer->capacity();
        setResizeThreshold();
 }
 
-// TODO: delete method
-synchronized void Table::printSlots() {
-       int64_t o = buffer.getOldestSeqNum();
-       int64_t n = buffer.getNewestSeqNum();
-
-       Array<int> *types = new Array<int>(10);
-
-       int num = 0;
-
-       int livec = 0;
-       int deadc = 0;
-
-       int casdasd = 0;
-
-       int liveslo = 0;
-
-       for (int64_t i = o; i < (n + 1); i++) {
-               Slot s = buffer.getSlot(i);
-
-
-               if (s.isLive()) {
-                       liveslo++;
-               }
-
-               Vector<Entry> entries = s.getEntries();
-
-               for (Entry e : entries) {
-                       if (e.isLive()) {
-                               int type = e.getType();
-
-
-                               if (type == 6) {
-                                       RejectedMessage rej = (RejectedMessage)e;
-                                       casdasd++;
-
-                                       System.out.println(rej.getMachineID());
-                               }
-
-
-                               types[type] = types[type] + 1;
-                               num++;
-                               livec++;
-                       } else {
-                               deadc++;
-                       }
-               }
-       }
-
-       for (int i = 0; i < 10; i++) {
-               System.out.println(i + "    " + types[i]);
-       }
-       System.out.println("Live count:   " + livec);
-       System.out.println("Live Slot count:   " + liveslo);
-
-       System.out.println("Dead count:   " + deadc);
-       System.out.println("Old:   " + o);
-       System.out.println("New:   " + n);
-       System.out.println("Size:   " + buffer.size());
-       // System.out.println("Commits:   " + liveCommitsTable.size());
-       System.out.println("pendingTrans:   " + pendingTransactionQueue.size());
-       System.out.println("Trans Status Out:   " + outstandingTransactionStatus.size());
-
-       for (Long k : lastArbitratedTransactionNumberByArbitratorTable.keySet()) {
-               System.out.println(k + ": " + lastArbitratedTransactionNumberByArbitratorTable.get(k));
-       }
-
-
-       for (Long a : liveCommitsTable.keySet()) {
-               for (Long b : liveCommitsTable.get(a).keySet()) {
-                       for (KeyValue kv : liveCommitsTable.get(a).get(b).getKeyValueUpdateSet()) {
-                               System.out.print(kv + " ");
-                       }
-                       System.out.print("|| ");
-               }
-               System.out.println();
-       }
-
-}
-
 /**
- * Initialize the table by inserting a table status as the first entry into the table status
- * also initialize the crypto stuff.
+ * Initialize the table by inserting a table status as the first entry
+ * into the table status also initialize the crypto stuff.
  */
-synchronized void Table::initTable() {
-       cloud.initSecurity();
+void Table::initTable() {
+       cloud->initSecurity();
 
        // Create the first insertion into the block chain which is the table status
-       Slot s = new Slot(this, 1, localMachineId, localSequenceNumber);
+       Slot *s = new Slot(this, 1, localMachineId, localSequenceNumber);
        localSequenceNumber++;
-       TableStatus status = new TableStatus(s, numberOfSlots);
-       s.addEntry(status);
-       Array<Slot> *array = cloud.putSlot(s, numberOfSlots);
+       TableStatus *status = new TableStatus(s, numberOfSlots);
+       s->addEntry(status);
+       Array<Slot *> *array = cloud->putSlot(s, numberOfSlots);
 
        if (array == NULL) {
-               array = new Array<Slot>(1);
+               array = new Array<Slot *>(1);
                array->set(0, s);
                // update local block chain
                validateAndUpdate(array, true);
-       } else if (array.length == 1) {
+       } else if (array->length() == 1) {
                // in case we did push the slot BUT we failed to init it
                validateAndUpdate(array, true);
        } else {
@@ -278,152 +226,135 @@ synchronized void Table::initTable() {
 }
 
 /**
- * Rebuild the table from scratch by pulling the latest block chain from the server.
+ * Rebuild the table from scratch by pulling the latest block chain
+ * from the server.
  */
-synchronized void Table::rebuild() {
+void Table::rebuild() {
        // Just pull the latest slots from the server
-       Array<Slot> *newslots = cloud.getSlots(sequenceNumber + 1);
+       Array<Slot *> *newslots = cloud->getSlots(sequenceNumber + 1);
        validateAndUpdate(newslots, true);
        sendToServer(NULL);
        updateLiveTransactionsAndStatus();
-
 }
 
-// String toString() {
-//  String retString = " Committed Table: \n";
-//  retString += "---------------------------\n";
-//  retString += commitedTable.toString();
-
-//  retString += "\n\n";
-
-//  retString += " Speculative Table: \n";
-//  retString += "---------------------------\n";
-//  retString += speculativeTable.toString();
-
-//  return retString;
-// }
-
-synchronized void Table::addLocalCommunication(int64_t arbitrator, String hostName, int portNumber) {
-       localCommunicationTable.put(arbitrator, new Pair<String, int32_t>(hostName, portNumber));
+void Table::addLocalCommunication(int64_t arbitrator, IoTString *hostName, int portNumber) {
+       localCommunicationTable->put(arbitrator, new Pair<IoTString *, int32_t>(hostName, portNumber));
 }
 
-synchronized Long Table::getArbitrator(IoTString key) {
-       return arbitratorTable.get(key);
+int64_t Table::getArbitrator(IoTString *key) {
+       return arbitratorTable->get(key);
 }
 
-synchronized void Table::close() {
-       cloud.close();
+void Table::close() {
+       cloud->closeCloud();
 }
 
-synchronized IoTString Table::getCommitted(IoTString key)  {
-       KeyValue kv = committedKeyValueTable.get(key);
+IoTString *Table::getCommitted(IoTString *key)  {
+       KeyValue *kv = committedKeyValueTable->get(key);
 
        if (kv != NULL) {
-               return kv.getValue();
+               return kv->getValue();
        } else {
                return NULL;
        }
 }
 
-synchronized IoTString Table::getSpeculative(IoTString key) {
-       KeyValue kv = pendingTransactionSpeculatedKeyValueTable.get(key);
+IoTString *Table::getSpeculative(IoTString *key) {
+       KeyValue *kv = pendingTransactionSpeculatedKeyValueTable->get(key);
 
        if (kv == NULL) {
-               kv = speculatedKeyValueTable.get(key);
+               kv = speculatedKeyValueTable->get(key);
        }
 
        if (kv == NULL) {
-               kv = committedKeyValueTable.get(key);
+               kv = committedKeyValueTable->get(key);
        }
 
        if (kv != NULL) {
-               return kv.getValue();
+               return kv->getValue();
        } else {
                return NULL;
        }
 }
 
-synchronized IoTString Table::getCommittedAtomic(IoTString key) {
-       KeyValue kv = committedKeyValueTable.get(key);
+IoTString *Table::getCommittedAtomic(IoTString *key) {
+       KeyValue *kv = committedKeyValueTable->get(key);
 
-       if (arbitratorTable.get(key) == NULL) {
+       if (!arbitratorTable->contains(key)) {
                throw new Error("Key not Found.");
        }
 
        // Make sure new key value pair matches the current arbitrator
-       if (!pendingTransactionBuilder.checkArbitrator(arbitratorTable.get(key))) {
+       if (!pendingTransactionBuilder->checkArbitrator(arbitratorTable->get(key))) {
                // TODO: Maybe not throw en error
                throw new Error("Not all Key Values Match Arbitrator.");
        }
 
        if (kv != NULL) {
-               pendingTransactionBuilder.addKVGuard(new KeyValue(key, kv.getValue()));
-               return kv.getValue();
+               pendingTransactionBuilder->addKVGuard(new KeyValue(key, kv->getValue()));
+               return kv->getValue();
        } else {
-               pendingTransactionBuilder.addKVGuard(new KeyValue(key, NULL));
+               pendingTransactionBuilder->addKVGuard(new KeyValue(key, NULL));
                return NULL;
        }
 }
 
-synchronized IoTString Table::getSpeculativeAtomic(IoTString key) {
-       if (arbitratorTable.get(key) == NULL) {
+IoTString *Table::getSpeculativeAtomic(IoTString *key) {
+       if (!arbitratorTable->contains(key)) {
                throw new Error("Key not Found.");
        }
 
        // Make sure new key value pair matches the current arbitrator
-       if (!pendingTransactionBuilder.checkArbitrator(arbitratorTable.get(key))) {
+       if (!pendingTransactionBuilder->checkArbitrator(arbitratorTable->get(key))) {
                // TODO: Maybe not throw en error
                throw new Error("Not all Key Values Match Arbitrator.");
        }
 
-       KeyValue kv = pendingTransactionSpeculatedKeyValueTable.get(key);
+       KeyValue *kv = pendingTransactionSpeculatedKeyValueTable->get(key);
 
        if (kv == NULL) {
-               kv = speculatedKeyValueTable.get(key);
+               kv = speculatedKeyValueTable->get(key);
        }
 
        if (kv == NULL) {
-               kv = committedKeyValueTable.get(key);
+               kv = committedKeyValueTable->get(key);
        }
 
        if (kv != NULL) {
-               pendingTransactionBuilder.addKVGuard(new KeyValue(key, kv.getValue()));
-               return kv.getValue();
+               pendingTransactionBuilder->addKVGuard(new KeyValue(key, kv->getValue()));
+               return kv->getValue();
        } else {
-               pendingTransactionBuilder.addKVGuard(new KeyValue(key, NULL));
+               pendingTransactionBuilder->addKVGuard(new KeyValue(key, NULL));
                return NULL;
        }
 }
 
-synchronized bool Table::update()  {
+bool Table::update()  {
        try {
-               Array<Slot> *newSlots = cloud.getSlots(sequenceNumber + 1);
+               Array<Slot *> *newSlots = cloud->getSlots(sequenceNumber + 1);
                validateAndUpdate(newSlots, false);
                sendToServer(NULL);
-
-
                updateLiveTransactionsAndStatus();
-
                return true;
-       } catch (Exception e) {
-               // e.printStackTrace();
-
-               for (Long m : localCommunicationTable.keySet()) {
+       } catch (Exception *e) {
+               SetIterator<int64_t, Pair<IoTString *, int32_t> *> *kit = getKeyIterator(localCommunicationTable);
+               while (kit->hasNext()) {
+                       int64_t m = kit->next();
                        updateFromLocal(m);
                }
+               delete kit;
        }
 
        return false;
 }
 
-synchronized bool Table::createNewKey(IoTString keyName, int64_t machineId) {
+bool Table::createNewKey(IoTString *keyName, int64_t machineId) {
        while (true) {
-               if (arbitratorTable.get(keyName) != NULL) {
+               if (!arbitratorTable->contains(keyName)) {
                        // There is already an arbitrator
                        return false;
                }
-
-               NewKey newKey = new NewKey(NULL, keyName, machineId);
+               NewKey *newKey = new NewKey(NULL, keyName, machineId);
 
                if (sendToServer(newKey)) {
                        // If successfully inserted
@@ -432,50 +363,49 @@ synchronized bool Table::createNewKey(IoTString keyName, int64_t machineId) {
        }
 }
 
-synchronized void Table::startTransaction() {
+void Table::startTransaction() {
        // Create a new transaction, invalidates any old pending transactions.
        pendingTransactionBuilder = new PendingTransaction(localMachineId);
 }
 
-synchronized void Table::addKV(IoTString key, IoTString value) {
+void Table::addKV(IoTString *key, IoTString *value) {
 
        // Make sure it is a valid key
-       if (arbitratorTable.get(key) == NULL) {
+       if (!arbitratorTable->contains(key)) {
                throw new Error("Key not Found.");
        }
 
        // Make sure new key value pair matches the current arbitrator
-       if (!pendingTransactionBuilder.checkArbitrator(arbitratorTable.get(key))) {
+       if (!pendingTransactionBuilder->checkArbitrator(arbitratorTable->get(key))) {
                // TODO: Maybe not throw en error
                throw new Error("Not all Key Values Match Arbitrator.");
        }
 
        // Add the key value to this transaction
-       KeyValue kv = new KeyValue(key, value);
-       pendingTransactionBuilder.addKV(kv);
+       KeyValue *kv = new KeyValue(key, value);
+       pendingTransactionBuilder->addKV(kv);
 }
 
-synchronized TransactionStatus Table::commitTransaction() {
-
-       if (pendingTransactionBuilder.getKVUpdates().size() == 0) {
+TransactionStatus *Table::commitTransaction() {
+       if (pendingTransactionBuilder->getKVUpdates()->size() == 0) {
                // transaction with no updates will have no effect on the system
-               return new TransactionStatus(TransactionStatus.StatusNoEffect, -1);
+               return new TransactionStatus(TransactionStatus_StatusNoEffect, -1);
        }
 
        // Set the local transaction sequence number and increment
-       pendingTransactionBuilder.setClientLocalSequenceNumber(localTransactionSequenceNumber);
+       pendingTransactionBuilder->setClientLocalSequenceNumber(localTransactionSequenceNumber);
        localTransactionSequenceNumber++;
 
        // Create the transaction status
-       TransactionStatus transactionStatus = new TransactionStatus(TransactionStatus.StatusPending, pendingTransactionBuilder.getArbitrator());
+       TransactionStatus *transactionStatus = new TransactionStatus(TransactionStatus_StatusPending, pendingTransactionBuilder->getArbitrator());
 
        // Create the new transaction
-       Transaction newTransaction = pendingTransactionBuilder.createTransaction();
-       newTransaction.setTransactionStatus(transactionStatus);
+       Transaction *newTransaction = pendingTransactionBuilder->createTransaction();
+       newTransaction->setTransactionStatus(transactionStatus);
 
-       if (pendingTransactionBuilder.getArbitrator() != localMachineId) {
+       if (pendingTransactionBuilder->getArbitrator() != localMachineId) {
                // Add it to the queue and invalidate the builder for safety
-               pendingTransactionQueue.add(newTransaction);
+               pendingTransactionQueue->add(newTransaction);
        } else {
                arbitrateOnLocalTransaction(newTransaction);
                updateLiveStateFromLocal();
@@ -485,13 +415,16 @@ synchronized TransactionStatus Table::commitTransaction() {
 
        try {
                sendToServer(NULL);
-       } catch (ServerException e) {
+       } catch (ServerException *e) {
 
-               Set<Long> arbitratorTriedAndFailed = new Hashset<Long>();
-               for (Iterator<Transaction> iter = pendingTransactionQueue.iterator(); iter.hasNext(); ) {
-                       Transaction transaction = iter.next();
+               Hashset<int64_t> *arbitratorTriedAndFailed = new Hashset<int64_t>();
+               uint size = pendingTransactionQueue->size();
+               uint oldindex = 0;
+               for (int iter = 0; iter < size; iter++) {
+                       Transaction *transaction = pendingTransactionQueue->get(iter);
+                       pendingTransactionQueue->set(oldindex++, pendingTransactionQueue->get(iter));
 
-                       if (arbitratorTriedAndFailed.contains(transaction.getArbitrator())) {
+                       if (arbitratorTriedAndFailed->contains(transaction->getArbitrator())) {
                                // Already contacted this client so ignore all attempts to contact this client
                                // to preserve ordering for arbitrator
                                continue;
@@ -501,16 +434,17 @@ synchronized TransactionStatus Table::commitTransaction() {
 
                        if (sendReturn.getFirst()) {
                                // Failed to contact over local
-                               arbitratorTriedAndFailed.add(transaction.getArbitrator());
+                               arbitratorTriedAndFailed->add(transaction->getArbitrator());
                        } else {
                                // Successful contact or should not contact
 
                                if (sendReturn.getSecond()) {
                                        // did arbitrate
-                                       iter.remove();
+                                       oldindex--;
                                }
                        }
                }
+               pendingTransactionQueue->setSize(oldindex);
        }
 
        updateLiveStateFromLocal();
@@ -518,94 +452,79 @@ synchronized TransactionStatus Table::commitTransaction() {
        return transactionStatus;
 }
 
-/**
- * Get the machine ID for this client
- */
-int64_t Table::getMachineId() {
-       return localMachineId;
-}
-
-/**
- * Decrement the number of live slots that we currently have
- */
-void Table::decrementLiveCount() {
-       liveSlotCount--;
-}
-
 /**
  * Recalculate the new resize threshold
  */
 void Table::setResizeThreshold() {
-       int resizeLower = (int) (RESIZE_THRESHOLD * numberOfSlots);
-       bufferResizeThreshold = resizeLower - 1 + random.nextInt(numberOfSlots - resizeLower);
+       int resizeLower = (int) (Table_RESIZE_THRESHOLD * numberOfSlots);
+       bufferResizeThreshold = resizeLower - 1 + random->nextInt(numberOfSlots - resizeLower);
 }
 
 int64_t Table::getLocalSequenceNumber() {
        return localSequenceNumber;
 }
 
-
-bool lastInsertedNewKey = false;
-
-bool Table::sendToServer(NewKey newKey) {
-
+bool Table::sendToServer(NewKey *newKey) {
        bool fromRetry = false;
-
        try {
                if (hadPartialSendToServer) {
-                       Array<Slot> *newSlots = cloud.getSlots(sequenceNumber + 1);
-                       if (newSlots.length == 0) {
+                       Array<Slot *> *newSlots = cloud->getSlots(sequenceNumber + 1);
+                       if (newSlots->length() == 0) {
                                fromRetry = true;
-                               ThreeTuple<bool, bool, Array<Slot> *> sendSlotsReturn = sendSlotsToServer(lastSlotAttemptedToSend, lastNewSize, lastIsNewKey);
+                               ThreeTuple<bool, bool, Array<Slot *> *> sendSlotsReturn = sendSlotsToServer(lastSlotAttemptedToSend, lastNewSize, lastIsNewKey);
 
                                if (sendSlotsReturn.getFirst()) {
                                        if (newKey != NULL) {
-                                               if (lastInsertedNewKey && (lastNewKey.getKey() == newKey.getKey()) && (lastNewKey.getMachineID() == newKey.getMachineID())) {
+                                               if (lastInsertedNewKey && (lastNewKey->getKey() == newKey->getKey()) && (lastNewKey->getMachineID() == newKey->getMachineID())) {
                                                        newKey = NULL;
                                                }
                                        }
 
-                                       for (Transaction transaction : lastTransactionPartsSent.keySet()) {
-                                               transaction.resetServerFailure();
-
+                                       SetIterator<Transaction *, Vector<int> *> *trit = getKeyIterator(lastTransactionPartsSent);
+                                       while (trit->hasNext()) {
+                                               Transaction *transaction = trit->next();
+                                               transaction->resetServerFailure();
                                                // Update which transactions parts still need to be sent
-                                               transaction.removeSentParts(lastTransactionPartsSent.get(transaction));
-
+                                               transaction->removeSentParts(lastTransactionPartsSent->get(transaction));
                                                // Add the transaction status to the outstanding list
-                                               outstandingTransactionStatus.put(transaction.getSequenceNumber(), transaction.getTransactionStatus());
+                                               outstandingTransactionStatus->put(transaction->getSequenceNumber(), transaction->getTransactionStatus());
 
                                                // Update the transaction status
-                                               transaction.getTransactionStatus().setStatus(TransactionStatus.StatusSentPartial);
+                                               transaction->getTransactionStatus()->setStatus(TransactionStatus_StatusSentPartial);
 
-                                               // Check if all the transaction parts were successfully sent and if so then remove it from pending
-                                               if (transaction.didSendAllParts()) {
-                                                       transaction.getTransactionStatus().setStatus(TransactionStatus.StatusSentFully);
-                                                       pendingTransactionQueue.remove(transaction);
+                                               // Check if all the transaction parts were successfully
+                                               // sent and if so then remove it from pending
+                                               if (transaction->didSendAllParts()) {
+                                                       transaction->getTransactionStatus()->setStatus(TransactionStatus_StatusSentFully);
+                                                       pendingTransactionQueue->remove(transaction);
                                                }
                                        }
+                                       delete trit;
                                } else {
-
                                        newSlots = sendSlotsReturn.getThird();
-
                                        bool isInserted = false;
-                                       for (Slot s : newSlots) {
-                                               if ((s.getSequenceNumber() == lastSlotAttemptedToSend.getSequenceNumber()) && (s.getMachineID() == localMachineId)) {
+                                       for (uint si = 0; si < newSlots->length(); si++) {
+                                               Slot *s = newSlots->get(si);
+                                               if ((s->getSequenceNumber() == lastSlotAttemptedToSend->getSequenceNumber()) && (s->getMachineID() == localMachineId)) {
                                                        isInserted = true;
                                                        break;
                                                }
                                        }
 
-                                       for (Slot s : newSlots) {
+                                       for (uint si = 0; si < newSlots->length(); si++) {
+                                               Slot *s = newSlots->get(si);
                                                if (isInserted) {
                                                        break;
                                                }
 
                                                // Process each entry in the slot
-                                               for (Entry entry : s.getEntries()) {
-
-                                                       if (entry.getType() == Entry.TypeLastMessage) {
-                                                               LastMessage lastMessage = (LastMessage)entry;
-                                                               if ((lastMessage.getMachineID() == localMachineId) && (lastMessage.getSequenceNumber() == lastSlotAttemptedToSend.getSequenceNumber())) {
+                                               Vector<Entry *> *ventries = s->getEntries();
+                                               uint vesize = ventries->size();
+                                               for (uint vei = 0; vei < vesize; vei++) {
+                                                       Entry *entry = ventries->get(vei);
+                                                       if (entry->getType() == TypeLastMessage) {
+                                                               LastMessage *lastMessage = (LastMessage *)entry;
+                                                               if ((lastMessage->getMachineID() == localMachineId) && (lastMessage->getSequenceNumber() == lastSlotAttemptedToSend->getSequenceNumber())) {
                                                                        isInserted = true;
                                                                        break;
                                                                }
@@ -615,71 +534,82 @@ bool Table::sendToServer(NewKey newKey) {
 
                                        if (isInserted) {
                                                if (newKey != NULL) {
-                                                       if (lastInsertedNewKey && (lastNewKey.getKey() == newKey.getKey()) && (lastNewKey.getMachineID() == newKey.getMachineID())) {
+                                                       if (lastInsertedNewKey && (lastNewKey->getKey() == newKey->getKey()) && (lastNewKey->getMachineID() == newKey->getMachineID())) {
                                                                newKey = NULL;
                                                        }
                                                }
 
-                                               for (Transaction transaction : lastTransactionPartsSent.keySet()) {
-                                                       transaction.resetServerFailure();
+                                               SetIterator<Transaction *, Vector<int> *> *trit = getKeyIterator(lastTransactionPartsSent);
+                                               while (trit->hasNext()) {
+                                                       Transaction *transaction = trit->next();
+                                                       transaction->resetServerFailure();
 
                                                        // Update which transactions parts still need to be sent
-                                                       transaction.removeSentParts(lastTransactionPartsSent.get(transaction));
+                                                       transaction->removeSentParts(lastTransactionPartsSent->get(transaction));
 
                                                        // Add the transaction status to the outstanding list
-                                                       outstandingTransactionStatus.put(transaction.getSequenceNumber(), transaction.getTransactionStatus());
+                                                       outstandingTransactionStatus->put(transaction->getSequenceNumber(), transaction->getTransactionStatus());
 
                                                        // Update the transaction status
-                                                       transaction.getTransactionStatus().setStatus(TransactionStatus.StatusSentPartial);
+                                                       transaction->getTransactionStatus()->setStatus(TransactionStatus_StatusSentPartial);
 
                                                        // Check if all the transaction parts were successfully sent and if so then remove it from pending
-                                                       if (transaction.didSendAllParts()) {
-                                                               transaction.getTransactionStatus().setStatus(TransactionStatus.StatusSentFully);
-                                                               pendingTransactionQueue.remove(transaction);
+                                                       if (transaction->didSendAllParts()) {
+                                                               transaction->getTransactionStatus()->setStatus(TransactionStatus_StatusSentFully);
+                                                               pendingTransactionQueue->remove(transaction);
                                                        } else {
-                                                               transaction.resetServerFailure();
+                                                               transaction->resetServerFailure();
                                                                // Set the transaction sequence number back to nothing
-                                                               if (!transaction.didSendAPartToServer()) {
-                                                                       transaction.setSequenceNumber(-1);
+                                                               if (!transaction->didSendAPartToServer()) {
+                                                                       transaction->setSequenceNumber(-1);
                                                                }
                                                        }
                                                }
+                                               delete trit;
                                        }
                                }
 
-                               for (Transaction transaction : lastTransactionPartsSent.keySet()) {
-                                       transaction.resetServerFailure();
+                               SetIterator<Transaction *, Vector<int> *> *trit = getKeyIterator(lastTransactionPartsSent);
+                               while (trit->hasNext()) {
+                                       Transaction *transaction = trit->next();
+                                       transaction->resetServerFailure();
                                        // Set the transaction sequence number back to nothing
-                                       if (!transaction.didSendAPartToServer()) {
-                                               transaction.setSequenceNumber(-1);
+                                       if (!transaction->didSendAPartToServer()) {
+                                               transaction->setSequenceNumber(-1);
                                        }
                                }
+                               delete trit;
 
-                               if (sendSlotsReturn.getThird().length != 0) {
+                               if (sendSlotsReturn.getThird()->length() != 0) {
                                        // insert into the local block chain
                                        validateAndUpdate(sendSlotsReturn.getThird(), true);
                                }
                                // continue;
                        } else {
                                bool isInserted = false;
-                               for (Slot s : newSlots) {
-                                       if ((s.getSequenceNumber() == lastSlotAttemptedToSend.getSequenceNumber()) && (s.getMachineID() == localMachineId)) {
+                               for (uint si = 0; si < newSlots->length(); si++) {
+                                       Slot *s = newSlots->get(si);
+                                       if ((s->getSequenceNumber() == lastSlotAttemptedToSend->getSequenceNumber()) && (s->getMachineID() == localMachineId)) {
                                                isInserted = true;
                                                break;
                                        }
                                }
 
-                               for (Slot s : newSlots) {
+                               for (uint si = 0; si < newSlots->length(); si++) {
+                                       Slot *s = newSlots->get(si);
                                        if (isInserted) {
                                                break;
                                        }
 
                                        // Process each entry in the slot
-                                       for (Entry entry : s.getEntries()) {
-
-                                               if (entry.getType() == Entry.TypeLastMessage) {
-                                                       LastMessage lastMessage = (LastMessage)entry;
-                                                       if ((lastMessage.getMachineID() == localMachineId) && (lastMessage.getSequenceNumber() == lastSlotAttemptedToSend.getSequenceNumber())) {
+                                       Vector<Entry *> *entries = s->getEntries();
+                                       uint eSize = entries->size();
+                                       for (uint ei = 0; ei < eSize; ei++) {
+                                               Entry *entry = entries->get(ei);
+
+                                               if (entry->getType() == TypeLastMessage) {
+                                                       LastMessage *lastMessage = (LastMessage *)entry;
+                                                       if ((lastMessage->getMachineID() == localMachineId) && (lastMessage->getSequenceNumber() == lastSlotAttemptedToSend->getSequenceNumber())) {
                                                                isInserted = true;
                                                                break;
                                                        }
@@ -689,50 +619,56 @@ bool Table::sendToServer(NewKey newKey) {
 
                                if (isInserted) {
                                        if (newKey != NULL) {
-                                               if (lastInsertedNewKey && (lastNewKey.getKey() == newKey.getKey()) && (lastNewKey.getMachineID() == newKey.getMachineID())) {
+                                               if (lastInsertedNewKey && (lastNewKey->getKey() == newKey->getKey()) && (lastNewKey->getMachineID() == newKey->getMachineID())) {
                                                        newKey = NULL;
                                                }
                                        }
 
-                                       for (Transaction transaction : lastTransactionPartsSent.keySet()) {
-                                               transaction.resetServerFailure();
+                                       SetIterator<Transaction *, Vector<int> *> *trit = getKeyIterator(lastTransactionPartsSent);
+                                       while (trit->hasNext()) {
+                                               Transaction *transaction = trit->next();
+                                               transaction->resetServerFailure();
 
                                                // Update which transactions parts still need to be sent
-                                               transaction.removeSentParts(lastTransactionPartsSent.get(transaction));
+                                               transaction->removeSentParts(lastTransactionPartsSent->get(transaction));
 
                                                // Add the transaction status to the outstanding list
-                                               outstandingTransactionStatus.put(transaction.getSequenceNumber(), transaction.getTransactionStatus());
+                                               outstandingTransactionStatus->put(transaction->getSequenceNumber(), transaction->getTransactionStatus());
 
                                                // Update the transaction status
-                                               transaction.getTransactionStatus().setStatus(TransactionStatus.StatusSentPartial);
+                                               transaction->getTransactionStatus()->setStatus(TransactionStatus_StatusSentPartial);
 
                                                // Check if all the transaction parts were successfully sent and if so then remove it from pending
-                                               if (transaction.didSendAllParts()) {
-                                                       transaction.getTransactionStatus().setStatus(TransactionStatus.StatusSentFully);
-                                                       pendingTransactionQueue.remove(transaction);
+                                               if (transaction->didSendAllParts()) {
+                                                       transaction->getTransactionStatus()->setStatus(TransactionStatus_StatusSentFully);
+                                                       pendingTransactionQueue->remove(transaction);
                                                } else {
-                                                       transaction.resetServerFailure();
+                                                       transaction->resetServerFailure();
                                                        // Set the transaction sequence number back to nothing
-                                                       if (!transaction.didSendAPartToServer()) {
-                                                               transaction.setSequenceNumber(-1);
+                                                       if (!transaction->didSendAPartToServer()) {
+                                                               transaction->setSequenceNumber(-1);
                                                        }
                                                }
                                        }
+                                       delete trit;
                                } else {
-                                       for (Transaction transaction : lastTransactionPartsSent.keySet()) {
-                                               transaction.resetServerFailure();
+                                       SetIterator<Transaction *, Vector<int> *> *trit = getKeyIterator(lastTransactionPartsSent);
+                                       while (trit->hasNext()) {
+                                               Transaction *transaction = trit->next();
+                                               transaction->resetServerFailure();
                                                // Set the transaction sequence number back to nothing
-                                               if (!transaction.didSendAPartToServer()) {
-                                                       transaction.setSequenceNumber(-1);
+                                               if (!transaction->didSendAPartToServer()) {
+                                                       transaction->setSequenceNumber(-1);
                                                }
                                        }
+                                       delete trit;
                                }
 
                                // insert into the local block chain
                                validateAndUpdate(newSlots, true);
                        }
                }
-       } catch (ServerException e) {
+       } catch (ServerException *e) {
                throw e;
        }
 
@@ -740,7 +676,7 @@ bool Table::sendToServer(NewKey newKey) {
 
        try {
                // While we have stuff that needs inserting into the block chain
-               while ((pendingTransactionQueue.size() > 0) || (pendingSendArbitrationRounds.size() > 0) || (newKey != NULL)) {
+               while ((pendingTransactionQueue->size() > 0) || (pendingSendArbitrationRounds->size() > 0) || (newKey != NULL)) {
 
                        fromRetry = false;
 
@@ -751,12 +687,12 @@ bool Table::sendToServer(NewKey newKey) {
 
 
                        // If there is a new key with same name then end
-                       if ((newKey != NULL) && (arbitratorTable.get(newKey.getKey()) != NULL)) {
+                       if ((newKey != NULL) && arbitratorTable->contains(newKey->getKey())) {
                                return false;
                        }
 
                        // Create the slot
-                       Slot slot = new Slot(this, sequenceNumber + 1, localMachineId, buffer.getSlot(sequenceNumber).getHMAC(), localSequenceNumber);
+                       Slot *slot = new Slot(this, sequenceNumber + 1, localMachineId, buffer->getSlot(sequenceNumber)->getHMAC(), localSequenceNumber);
                        localSequenceNumber++;
 
                        // Try to fill the slot with data
@@ -767,18 +703,21 @@ bool Table::sendToServer(NewKey newKey) {
 
                        if (needsResize) {
                                // Reset which transaction to send
-                               for (Transaction transaction : transactionPartsSent.keySet()) {
-                                       transaction.resetNextPartToSend();
+                               SetIterator<Transaction *, Vector<int> *> *trit = getKeyIterator(transactionPartsSent);
+                               while (trit->hasNext()) {
+                                       Transaction *transaction = trit->next();
+                                       transaction->resetNextPartToSend();
 
                                        // Set the transaction sequence number back to nothing
-                                       if (!transaction.didSendAPartToServer() && !transaction.getServerFailure()) {
-                                               transaction.setSequenceNumber(-1);
+                                       if (!transaction->didSendAPartToServer() && !transaction->getServerFailure()) {
+                                               transaction->setSequenceNumber(-1);
                                        }
                                }
+                               delete trit;
 
                                // Clear the sent data since we are trying again
-                               pendingSendArbitrationEntriesToDelete.clear();
-                               transactionPartsSent.clear();
+                               pendingSendArbitrationEntriesToDelete->clear();
+                               transactionPartsSent->clear();
 
                                // We needed a resize so try again
                                fillSlot(slot, true, newKey);
@@ -789,11 +728,10 @@ bool Table::sendToServer(NewKey newKey) {
                        lastInsertedNewKey = insertedNewKey;
                        lastNewSize = newSize;
                        lastNewKey = newKey;
-                       lastTransactionPartsSent = new Hashtable<Transaction, Vector<int32_t> >(transactionPartsSent);
-                       lastPendingSendArbitrationEntriesToDelete = new Vector<Entry>(pendingSendArbitrationEntriesToDelete);
+                       lastTransactionPartsSent = transactionPartsSent->clone();
+                       lastPendingSendArbitrationEntriesToDelete = new Vector<Entry *>(pendingSendArbitrationEntriesToDelete);
 
-
-                       ThreeTuple<bool, bool, Array<Slot> *> sendSlotsReturn = sendSlotsToServer(slot, newSize, newKey != NULL);
+                       ThreeTuple<bool, bool, Array<Slot *> *> sendSlotsReturn = sendSlotsToServer(slot, newSize, newKey != NULL);
 
                        if (sendSlotsReturn.getFirst()) {
 
@@ -807,120 +745,96 @@ bool Table::sendToServer(NewKey newKey) {
                                }
 
                                // Remove the aborts and commit parts that were sent from the pending to send queue
-                               for (Iterator<ArbitrationRound> iter = pendingSendArbitrationRounds.iterator(); iter.hasNext(); ) {
-                                       ArbitrationRound round = iter.next();
-                                       round.removeParts(pendingSendArbitrationEntriesToDelete);
+                               uint size = pendingSendArbitrationRounds->size();
+                               uint oldcount = 0;
+                               for (uint i = 0; i < size; i++) {
+                                       ArbitrationRound *round = pendingSendArbitrationRounds->get(i);
+                                       round->removeParts(pendingSendArbitrationEntriesToDelete);
 
-                                       if (round.isDoneSending()) {
+                                       if (!round->isDoneSending()) {
                                                // Sent all the parts
-                                               iter.remove();
+                                               pendingSendArbitrationRounds->set(oldcount++,
+                                                                                                                                                                                       pendingSendArbitrationRounds->get(i));
                                        }
                                }
+                               pendingSendArbitrationRounds->setSize(oldcount);
 
-                               for (Transaction transaction : transactionPartsSent.keySet()) {
-                                       transaction.resetServerFailure();
+                               SetIterator<Transaction *, Vector<int> *> *trit = getKeyIterator(transactionPartsSent);
+                               while (trit->hasNext()) {
+                                       Transaction *transaction = trit->next();
+                                       transaction->resetServerFailure();
 
                                        // Update which transactions parts still need to be sent
-                                       transaction.removeSentParts(transactionPartsSent.get(transaction));
+                                       transaction->removeSentParts(transactionPartsSent->get(transaction));
 
                                        // Add the transaction status to the outstanding list
-                                       outstandingTransactionStatus.put(transaction.getSequenceNumber(), transaction.getTransactionStatus());
+                                       outstandingTransactionStatus->put(transaction->getSequenceNumber(), transaction->getTransactionStatus());
 
                                        // Update the transaction status
-                                       transaction.getTransactionStatus().setStatus(TransactionStatus.StatusSentPartial);
+                                       transaction->getTransactionStatus()->setStatus(TransactionStatus_StatusSentPartial);
 
                                        // Check if all the transaction parts were successfully sent and if so then remove it from pending
-                                       if (transaction.didSendAllParts()) {
-                                               transaction.getTransactionStatus().setStatus(TransactionStatus.StatusSentFully);
-                                               pendingTransactionQueue.remove(transaction);
+                                       if (transaction->didSendAllParts()) {
+                                               transaction->getTransactionStatus()->setStatus(TransactionStatus_StatusSentFully);
+                                               pendingTransactionQueue->remove(transaction);
                                        }
                                }
+                               delete trit;
                        } else {
-
-                               // if (!sendSlotsReturn.getSecond()) {
-                               //  for (Transaction transaction : lastTransactionPartsSent.keySet()) {
-                               //    transaction.resetServerFailure();
-                               //  }
-                               // } else {
-                               //  for (Transaction transaction : lastTransactionPartsSent.keySet()) {
-                               //    transaction.resetServerFailure();
-
-                               //    // Update which transactions parts still need to be sent
-                               //    transaction.removeSentParts(transactionPartsSent.get(transaction));
-
-                               //    // Add the transaction status to the outstanding list
-                               //    outstandingTransactionStatus.put(transaction.getSequenceNumber(), transaction.getTransactionStatus());
-
-                               //    // Update the transaction status
-                               //    transaction.getTransactionStatus().setStatus(TransactionStatus.StatusSentPartial);
-
-                               //    // Check if all the transaction parts were successfully sent and if so then remove it from pending
-                               //    if (transaction.didSendAllParts()) {
-                               //      transaction.getTransactionStatus().setStatus(TransactionStatus.StatusSentFully);
-                               //      pendingTransactionQueue.remove(transaction);
-
-                               //      for (KeyValue kv : transaction.getKeyValueUpdateSet()) {
-                               //        System.out.println("Sent: " + kv + "  from: " + localMachineId + "   Slot:" + lastSlotAttemptedToSend.getSequenceNumber() + "  Claimed:" + transaction.getSequenceNumber());
-                               //      }
-                               //    }
-                               //  }
-                               // }
-
                                // Reset which transaction to send
-                               for (Transaction transaction : transactionPartsSent.keySet()) {
-                                       transaction.resetNextPartToSend();
-                                       // transaction.resetNextPartToSend();
+                               SetIterator<Transaction *, Vector<int> *> *trit = getKeyIterator(transactionPartsSent);
+                               while (trit->hasNext()) {
+                                       Transaction *transaction = trit->next();
+                                       transaction->resetNextPartToSend();
 
                                        // Set the transaction sequence number back to nothing
-                                       if (!transaction.didSendAPartToServer() && !transaction.getServerFailure()) {
-                                               transaction.setSequenceNumber(-1);
+                                       if (!transaction->didSendAPartToServer() && !transaction->getServerFailure()) {
+                                               transaction->setSequenceNumber(-1);
                                        }
                                }
+                               delete trit;
                        }
 
                        // Clear the sent data in preparation for next send
-                       pendingSendArbitrationEntriesToDelete.clear();
-                       transactionPartsSent.clear();
+                       pendingSendArbitrationEntriesToDelete->clear();
+                       transactionPartsSent->clear();
 
-                       if (sendSlotsReturn.getThird().length != 0) {
+                       if (sendSlotsReturn.getThird()->length() != 0) {
                                // insert into the local block chain
                                validateAndUpdate(sendSlotsReturn.getThird(), true);
                        }
                }
 
-       } catch (ServerException e) {
-
-               if (e.getType() != ServerException.TypeInputTimeout) {
-                       // e.printStackTrace();
-
+       } catch (ServerException *e) {
+               if (e->getType() != ServerException_TypeInputTimeout) {
                        // Nothing was able to be sent to the server so just clear these data structures
-                       for (Transaction transaction : transactionPartsSent.keySet()) {
-                               transaction.resetNextPartToSend();
+                       SetIterator<Transaction *, Vector<int> *> *trit = getKeyIterator(transactionPartsSent);
+                       while (trit->hasNext()) {
+                               Transaction *transaction = trit->next();
+                               transaction->resetNextPartToSend();
 
                                // Set the transaction sequence number back to nothing
-                               if (!transaction.didSendAPartToServer() && !transaction.getServerFailure()) {
-                                       transaction.setSequenceNumber(-1);
+                               if (!transaction->didSendAPartToServer() && !transaction->getServerFailure()) {
+                                       transaction->setSequenceNumber(-1);
                                }
                        }
+                       delete trit;
                } else {
                        // There was a partial send to the server
                        hadPartialSendToServer = true;
 
-
-                       // if (!fromRetry) {
-                       //  lastTransactionPartsSent = new Hashtable<Transaction, Vector<int32_t>>(transactionPartsSent);
-                       //  lastPendingSendArbitrationEntriesToDelete = new Vector<Entry>(pendingSendArbitrationEntriesToDelete);
-                       // }
-
                        // Nothing was able to be sent to the server so just clear these data structures
-                       for (Transaction transaction : transactionPartsSent.keySet()) {
-                               transaction.resetNextPartToSend();
-                               transaction.setServerFailure();
+                       SetIterator<Transaction *, Vector<int> *> *trit = getKeyIterator(transactionPartsSent);
+                       while (trit->hasNext()) {
+                               Transaction *transaction = trit->next();
+                               transaction->resetNextPartToSend();
+                               transaction->setServerFailure();
                        }
+                       delete trit;
                }
 
-               pendingSendArbitrationEntriesToDelete.clear();
-               transactionPartsSent.clear();
+               pendingSendArbitrationEntriesToDelete->clear();
+               transactionPartsSent->clear();
 
                throw e;
        }
@@ -928,30 +842,29 @@ bool Table::sendToServer(NewKey newKey) {
        return newKey == NULL;
 }
 
-synchronized bool Table::updateFromLocal(int64_t machineId) {
-       Pair<String, int32_t> localCommunicationInformation = localCommunicationTable.get(machineId);
-       if (localCommunicationInformation == NULL) {
-               // Cant talk to that device locally so do nothing
+bool Table::updateFromLocal(int64_t machineId) {
+       if (!localCommunicationTable->contains(machineId))
                return false;
-       }
+
+       Pair<IoTString *, int32_t> *localCommunicationInformation = localCommunicationTable->get(machineId);
 
        // Get the size of the send data
        int sendDataSize = sizeof(int32_t) + sizeof(int64_t);
 
-       Long lastArbitrationDataLocalSequenceNumber = (int64_t) -1;
-       if (lastArbitrationDataLocalSequenceNumberSeenFromArbitrator.get(machineId) != NULL) {
-               lastArbitrationDataLocalSequenceNumber = lastArbitrationDataLocalSequenceNumberSeenFromArbitrator.get(machineId);
+       int64_t lastArbitrationDataLocalSequenceNumber = (int64_t) -1;
+       if (lastArbitrationDataLocalSequenceNumberSeenFromArbitrator->contains(machineId)) {
+               lastArbitrationDataLocalSequenceNumber = lastArbitrationDataLocalSequenceNumberSeenFromArbitrator->get(machineId);
        }
 
-       Array<char> *sendData = new char[sendDataSize];
-       ByteBuffer bbEncode = ByteBuffer.wrap(sendData);
+       Array<char> *sendData = new Array<char>(sendDataSize);
+       ByteBuffer *bbEncode = ByteBuffer_wrap(sendData);
 
        // Encode the data
-       bbEncode.putLong(lastArbitrationDataLocalSequenceNumber);
-       bbEncode.putInt(0);
+       bbEncode->putLong(lastArbitrationDataLocalSequenceNumber);
+       bbEncode->putInt(0);
 
        // Send by local
-       Array<char> *returnData = cloud.sendLocalData(sendData, localSequenceNumber, localCommunicationInformation.getFirst(), localCommunicationInformation.getSecond());
+       Array<char> *returnData = cloud->sendLocalData(sendData, localSequenceNumber, localCommunicationInformation->getFirst(), localCommunicationInformation->getSecond());
        localSequenceNumber++;
 
        if (returnData == NULL) {
@@ -960,16 +873,16 @@ synchronized bool Table::updateFromLocal(int64_t machineId) {
        }
 
        // Decode the data
-       ByteBuffer bbDecode = ByteBuffer.wrap(returnData);
-       int numberOfEntries = bbDecode.getInt();
+       ByteBuffer *bbDecode = ByteBuffer_wrap(returnData);
+       int numberOfEntries = bbDecode->getInt();
 
        for (int i = 0; i < numberOfEntries; i++) {
-               char type = bbDecode.get();
-               if (type == Entry.TypeAbort) {
-                       Abort abort = (Abort)Abort.decode(NULL, bbDecode);
+               char type = bbDecode->get();
+               if (type == TypeAbort) {
+                       Abort *abort = (Abort *)Abort_decode(NULL, bbDecode);
                        processEntry(abort);
-               } else if (type == Entry.TypeCommitPart) {
-                       CommitPart commitPart = (CommitPart)CommitPart.decode(NULL, bbDecode);
+               } else if (type == TypeCommitPart) {
+                       CommitPart *commitPart = (CommitPart *)CommitPart_decode(NULL, bbDecode);
                        processEntry(commitPart);
                }
        }
@@ -979,67 +892,74 @@ synchronized bool Table::updateFromLocal(int64_t machineId) {
        return true;
 }
 
-Pair<bool, bool> Table::sendTransactionToLocal(Transaction transaction) {
+Pair<bool, bool> Table::sendTransactionToLocal(Transaction *transaction) {
 
        // Get the devices local communications
-       Pair<String, int32_t> localCommunicationInformation = localCommunicationTable.get(transaction.getArbitrator());
+       if (!localCommunicationTable->contains(transaction->getArbitrator()))
+               return Pair<bool, bool>(true, false);
 
-       if (localCommunicationInformation == NULL) {
-               // Cant talk to that device locally so do nothing
-               return new Pair<bool, bool>(true, false);
-       }
+       Pair<IoTString *, int32_t> *localCommunicationInformation = localCommunicationTable->get(transaction->getArbitrator());
 
        // Get the size of the send data
        int sendDataSize = sizeof(int32_t) + sizeof(int64_t);
-       for (TransactionPart part : transaction.getParts().values()) {
-               sendDataSize += part.getSize();
+       {
+               Vector<TransactionPart *> *tParts = transaction->getParts();
+               uint tPartsSize = tParts->size();
+               for (uint i = 0; i < tPartsSize; i++) {
+                       TransactionPart *part = tParts->get(i);
+                       sendDataSize += part->getSize();
+               }
        }
 
-       Long lastArbitrationDataLocalSequenceNumber = (int64_t) -1;
-       if (lastArbitrationDataLocalSequenceNumberSeenFromArbitrator.get(transaction.getArbitrator()) != NULL) {
-               lastArbitrationDataLocalSequenceNumber = lastArbitrationDataLocalSequenceNumberSeenFromArbitrator.get(transaction.getArbitrator());
+       int64_t lastArbitrationDataLocalSequenceNumber = (int64_t) -1;
+       if (lastArbitrationDataLocalSequenceNumberSeenFromArbitrator->contains(transaction->getArbitrator())) {
+               lastArbitrationDataLocalSequenceNumber = lastArbitrationDataLocalSequenceNumberSeenFromArbitrator->get(transaction->getArbitrator());
        }
 
        // Make the send data size
-       Array<char> *sendData = new char[sendDataSize];
-       ByteBuffer bbEncode = ByteBuffer.wrap(sendData);
+       Array<char> *sendData = new Array<char>(sendDataSize);
+       ByteBuffer *bbEncode = ByteBuffer_wrap(sendData);
 
        // Encode the data
-       bbEncode.putLong(lastArbitrationDataLocalSequenceNumber);
-       bbEncode.putInt(transaction.getParts().size());
-       for (TransactionPart part : transaction.getParts().values()) {
-               part.encode(bbEncode);
+       bbEncode->putLong(lastArbitrationDataLocalSequenceNumber);
+       bbEncode->putInt(transaction->getParts()->size());
+       {
+               Vector<TransactionPart *> *tParts = transaction->getParts();
+               uint tPartsSize = tParts->size();
+               for (uint i = 0; i < tPartsSize; i++) {
+                       TransactionPart *part = tParts->get(i);
+                       part->encode(bbEncode);
+               }
        }
 
-
        // Send by local
-       Array<char> *returnData = cloud.sendLocalData(sendData, localSequenceNumber, localCommunicationInformation.getFirst(), localCommunicationInformation.getSecond());
+       Array<char> *returnData = cloud->sendLocalData(sendData, localSequenceNumber, localCommunicationInformation->getFirst(), localCommunicationInformation->getSecond());
        localSequenceNumber++;
 
        if (returnData == NULL) {
                // Could not contact server
-               return new Pair<bool, bool>(true, false);
+               return Pair<bool, bool>(true, false);
        }
 
        // Decode the data
-       ByteBuffer bbDecode = ByteBuffer.wrap(returnData);
-       bool didCommit = bbDecode.get() == 1;
-       bool couldArbitrate = bbDecode.get() == 1;
-       int numberOfEntries = bbDecode.getInt();
+       ByteBuffer *bbDecode = ByteBuffer_wrap(returnData);
+       bool didCommit = bbDecode->get() == 1;
+       bool couldArbitrate = bbDecode->get() == 1;
+       int numberOfEntries = bbDecode->getInt();
        bool foundAbort = false;
 
        for (int i = 0; i < numberOfEntries; i++) {
-               char type = bbDecode.get();
-               if (type == Entry.TypeAbort) {
-                       Abort abort = (Abort)Abort.decode(NULL, bbDecode);
+               char type = bbDecode->get();
+               if (type == TypeAbort) {
+                       Abort *abort = (Abort *)Abort_decode(NULL, bbDecode);
 
-                       if ((abort.getTransactionMachineId() == localMachineId) && (abort.getTransactionClientLocalSequenceNumber() == transaction.getClientLocalSequenceNumber())) {
+                       if ((abort->getTransactionMachineId() == localMachineId) && (abort->getTransactionClientLocalSequenceNumber() == transaction->getClientLocalSequenceNumber())) {
                                foundAbort = true;
                        }
 
                        processEntry(abort);
-               } else if (type == Entry.TypeCommitPart) {
-                       CommitPart commitPart = (CommitPart)CommitPart.decode(NULL, bbDecode);
+               } else if (type == TypeCommitPart) {
+                       CommitPart *commitPart = (CommitPart *)CommitPart_decode(NULL, bbDecode);
                        processEntry(commitPart);
                }
        }
@@ -1047,30 +967,29 @@ Pair<bool, bool> Table::sendTransactionToLocal(Transaction transaction) {
        updateLiveStateFromLocal();
 
        if (couldArbitrate) {
-               TransactionStatus status =  transaction.getTransactionStatus();
+               TransactionStatus *status =  transaction->getTransactionStatus();
                if (didCommit) {
-                       status.setStatus(TransactionStatus.StatusCommitted);
+                       status->setStatus(TransactionStatus_StatusCommitted);
                } else {
-                       status.setStatus(TransactionStatus.StatusAborted);
+                       status->setStatus(TransactionStatus_StatusAborted);
                }
        } else {
-               TransactionStatus status =  transaction.getTransactionStatus();
+               TransactionStatus *status =  transaction->getTransactionStatus();
                if (foundAbort) {
-                       status.setStatus(TransactionStatus.StatusAborted);
+                       status->setStatus(TransactionStatus_StatusAborted);
                } else {
-                       status.setStatus(TransactionStatus.StatusCommitted);
+                       status->setStatus(TransactionStatus_StatusCommitted);
                }
        }
 
-       return new Pair<bool, bool>(false, true);
+       return Pair<bool, bool>(false, true);
 }
 
-synchronized Array<char> *Table::acceptDataFromLocal(Array<char> *data) {
-
+Array<char> *Table::acceptDataFromLocal(Array<char> *data) {
        // Decode the data
-       ByteBuffer bbDecode = ByteBuffer.wrap(data);
-       int64_t lastArbitratedSequenceNumberSeen = bbDecode.getLong();
-       int numberOfParts = bbDecode.getInt();
+       ByteBuffer *bbDecode = ByteBuffer_wrap(data);
+       int64_t lastArbitratedSequenceNumberSeen = bbDecode->getLong();
+       int numberOfParts = bbDecode->getInt();
 
        // If we did commit a transaction or not
        bool didCommit = false;
@@ -1079,11 +998,11 @@ synchronized Array<char> *Table::acceptDataFromLocal(Array<char> *data) {
        if (numberOfParts != 0) {
 
                // decode the transaction
-               Transaction transaction = new Transaction();
+               Transaction *transaction = new Transaction();
                for (int i = 0; i < numberOfParts; i++) {
-                       bbDecode.get();
-                       TransactionPart newPart = (TransactionPart)TransactionPart.decode(NULL, bbDecode);
-                       transaction.addPartDecode(newPart);
+                       bbDecode->get();
+                       TransactionPart *newPart = (TransactionPart *)TransactionPart_decode(NULL, bbDecode);
+                       transaction->addPartDecode(newPart);
                }
 
                // Arbitrate on transaction and pull relevant return data
@@ -1094,45 +1013,67 @@ synchronized Array<char> *Table::acceptDataFromLocal(Array<char> *data) {
                updateLiveStateFromLocal();
 
                // Transaction was sent to the server so keep track of it to prevent double commit
-               if (transaction.getSequenceNumber() != -1) {
-                       offlineTransactionsCommittedAndAtServer.add(transaction.getId());
+               if (transaction->getSequenceNumber() != -1) {
+                       offlineTransactionsCommittedAndAtServer->add(new Pair<int64_t, int64_t>(transaction->getId()));
                }
        }
 
        // The data to send back
        int returnDataSize = 0;
-       Vector<Entry> unseenArbitrations = new Vector<Entry>();
+       Vector<Entry *> *unseenArbitrations = new Vector<Entry *>();
 
        // Get the aborts to send back
-       Vector<Long> abortLocalSequenceNumbers = new Vector<Long >(liveAbortsGeneratedByLocal.keySet());
-       Collections.sort(abortLocalSequenceNumbers);
-       for (Long localSequenceNumber : abortLocalSequenceNumbers) {
+       Vector<int64_t> *abortLocalSequenceNumbers = new Vector<int64_t>();
+       {
+               SetIterator<int64_t, Abort *> *abortit = getKeyIterator(liveAbortsGeneratedByLocal);
+               while (abortit->hasNext())
+                       abortLocalSequenceNumbers->add(abortit->next());
+               delete abortit;
+       }
+
+       qsort(abortLocalSequenceNumbers->expose(), abortLocalSequenceNumbers->size(), sizeof(int64_t), compareInt64);
+
+       uint asize = abortLocalSequenceNumbers->size();
+       for (uint i = 0; i < asize; i++) {
+               int64_t localSequenceNumber = abortLocalSequenceNumbers->get(i);
                if (localSequenceNumber <= lastArbitratedSequenceNumberSeen) {
                        continue;
                }
 
-               Abort abort = liveAbortsGeneratedByLocal.get(localSequenceNumber);
-               unseenArbitrations.add(abort);
-               returnDataSize += abort.getSize();
+               Abort *abort = liveAbortsGeneratedByLocal->get(localSequenceNumber);
+               unseenArbitrations->add(abort);
+               returnDataSize += abort->getSize();
        }
 
        // Get the commits to send back
-       Hashtable<int64_t Commit> commitForClientTable = liveCommitsTable.get(localMachineId);
+       Hashtable<int64_t, Commit *> *commitForClientTable = liveCommitsTable->get(localMachineId);
        if (commitForClientTable != NULL) {
-               Vector<Long> commitLocalSequenceNumbers = new Vector<Long>(commitForClientTable.keySet());
-               Collections.sort(commitLocalSequenceNumbers);
+               Vector<int64_t> *commitLocalSequenceNumbers = new Vector<int64_t>();
+               {
+                       SetIterator<int64_t, Commit *> *commitit = getKeyIterator(commitForClientTable);
+                       while (commitit->hasNext())
+                               commitLocalSequenceNumbers->add(commitit->next());
+                       delete commitit;
+               }
+               qsort(commitLocalSequenceNumbers->expose(), commitLocalSequenceNumbers->size(), sizeof(int64_t), compareInt64);
 
-               for (Long localSequenceNumber : commitLocalSequenceNumbers) {
-                       Commit commit = commitForClientTable.get(localSequenceNumber);
+               uint clsSize = commitLocalSequenceNumbers->size();
+               for (uint clsi = 0; clsi < clsSize; clsi++) {
+                       int64_t localSequenceNumber = commitLocalSequenceNumbers->get(clsi);
+                       Commit *commit = commitForClientTable->get(localSequenceNumber);
 
                        if (localSequenceNumber <= lastArbitratedSequenceNumberSeen) {
                                continue;
                        }
 
-                       unseenArbitrations.addAll(commit.getParts().values());
-
-                       for (CommitPart commitPart : commit.getParts().values()) {
-                               returnDataSize += commitPart.getSize();
+                       {
+                               Vector<CommitPart *> *parts = commit->getParts();
+                               uint nParts = parts->size();
+                               for (uint i = 0; i < nParts; i++) {
+                                       CommitPart *commitPart = parts->get(i);
+                                       unseenArbitrations->add(commitPart);
+                                       returnDataSize += commitPart->getSize();
+                               }
                        }
                }
        }
@@ -1146,47 +1087,48 @@ synchronized Array<char> *Table::acceptDataFromLocal(Array<char> *data) {
        }
 
        // Data to send Back
-       Array<char> *returnData = new char[returnDataSize];
-       ByteBuffer bbEncode = ByteBuffer.wrap(returnData);
+       Array<char> *returnData = new Array<char>(returnDataSize);
+       ByteBuffer *bbEncode = ByteBuffer_wrap(returnData);
 
        if (numberOfParts != 0) {
                if (didCommit) {
-                       bbEncode.put((char)1);
+                       bbEncode->put((char)1);
                } else {
-                       bbEncode.put((char)0);
+                       bbEncode->put((char)0);
                }
                if (couldArbitrate) {
-                       bbEncode.put((char)1);
+                       bbEncode->put((char)1);
                } else {
-                       bbEncode.put((char)0);
+                       bbEncode->put((char)0);
                }
        }
 
-       bbEncode.putInt(unseenArbitrations.size());
-       for (Entry entry : unseenArbitrations) {
-               entry.encode(bbEncode);
+       bbEncode->putInt(unseenArbitrations->size());
+       uint size = unseenArbitrations->size();
+       for (uint i = 0; i < size; i++) {
+               Entry *entry = unseenArbitrations->get(i);
+               entry->encode(bbEncode);
        }
 
-
        localSequenceNumber++;
        return returnData;
 }
 
-ThreeTuple<bool, bool, Array<Slot> *> Table::sendSlotsToServer(Slot slot, int newSize, bool isNewKey) {
+ThreeTuple<bool, bool, Array<Slot *> *> Table::sendSlotsToServer(Slot *slot, int newSize, bool isNewKey) {
        bool attemptedToSendToServerTmp = attemptedToSendToServer;
        attemptedToSendToServer = true;
 
        bool inserted = false;
        bool lastTryInserted = false;
 
-       Array<Slot> *array = cloud.putSlot(slot, newSize);
+       Array<Slot *> *array = cloud->putSlot(slot, newSize);
        if (array == NULL) {
-               array = new Array<Slot>();
+               array = new Array<Slot *>();
                array->set(0, slot);
-               rejectedSlotVector.clear();
+               rejectedSlotVector->clear();
                inserted = true;
        } else {
-               if (array.length == 0) {
+               if (array->length() == 0) {
                        throw new Error("Server Error: Did not send any slots");
                }
 
@@ -1194,25 +1136,31 @@ ThreeTuple<bool, bool, Array<Slot> *> Table::sendSlotsToServer(Slot slot, int ne
                if (hadPartialSendToServer) {
 
                        bool isInserted = false;
-                       for (Slot s : array) {
-                               if ((s.getSequenceNumber() == slot.getSequenceNumber()) && (s.getMachineID() == localMachineId)) {
+                       uint size = array->length();
+                       for (uint i = 0; i < size; i++) {
+                               Slot *s = array->get(i);
+                               if ((s->getSequenceNumber() == slot->getSequenceNumber()) && (s->getMachineID() == localMachineId)) {
                                        isInserted = true;
                                        break;
                                }
                        }
 
-                       for (Slot s : array) {
+                       for (uint i = 0; i < size; i++) {
+                               Slot *s = array->get(i);
                                if (isInserted) {
                                        break;
                                }
 
                                // Process each entry in the slot
-                               for (Entry entry : s.getEntries()) {
+                               Vector<Entry *> *entries = s->getEntries();
+                               uint eSize = entries->size();
+                               for (uint ei = 0; ei < eSize; ei++) {
+                                       Entry *entry = entries->get(ei);
 
-                                       if (entry.getType() == Entry.TypeLastMessage) {
-                                               LastMessage lastMessage = (LastMessage)entry;
+                                       if (entry->getType() == TypeLastMessage) {
+                                               LastMessage *lastMessage = (LastMessage *)entry;
 
-                                               if ((lastMessage.getMachineID() == localMachineId) && (lastMessage.getSequenceNumber() == slot.getSequenceNumber())) {
+                                               if ((lastMessage->getMachineID() == localMachineId) && (lastMessage->getSequenceNumber() == slot->getSequenceNumber())) {
                                                        isInserted = true;
                                                        break;
                                                }
@@ -1221,43 +1169,40 @@ ThreeTuple<bool, bool, Array<Slot> *> Table::sendSlotsToServer(Slot slot, int ne
                        }
 
                        if (!isInserted) {
-                               rejectedSlotVector.add(slot.getSequenceNumber());
+                               rejectedSlotVector->add(slot->getSequenceNumber());
                                lastTryInserted = false;
                        } else {
                                lastTryInserted = true;
                        }
                } else {
-                       rejectedSlotVector.add(slot.getSequenceNumber());
+                       rejectedSlotVector->add(slot->getSequenceNumber());
                        lastTryInserted = false;
                }
        }
 
-       return new ThreeTuple<bool, bool, Array<Slot> *>(inserted, lastTryInserted, array);
+       return ThreeTuple<bool, bool, Array<Slot *> *>(inserted, lastTryInserted, array);
 }
 
 /**
  * Returns false if a resize was needed
  */
-ThreeTuple<bool, int32_t, bool> *Table::fillSlot(Slot slot, bool resize, NewKey newKeyEntry) {
-
-
+ThreeTuple<bool, int32_t, bool> Table::fillSlot(Slot *slot, bool resize, NewKey *newKeyEntry) {
        int newSize = 0;
        if (liveSlotCount > bufferResizeThreshold) {
-               resize = true;  //Resize is forced
-
+               resize = true;//Resize is forced
        }
 
        if (resize) {
-               newSize = (int) (numberOfSlots * RESIZE_MULTIPLE);
-               TableStatus status = new TableStatus(slot, newSize);
-               slot.addEntry(status);
+               newSize = (int) (numberOfSlots * Table_RESIZE_MULTIPLE);
+               TableStatus *status = new TableStatus(slot, newSize);
+               slot->addEntry(status);
        }
 
        // Fill with rejected slots first before doing anything else
        doRejectedMessages(slot);
 
        // Do mandatory rescue of entries
-       ThreeTuple<bool, bool, Long> mandatoryRescueReturn = doMandatoryResuce(slot, resize);
+       ThreeTuple<bool, bool, int64_t> mandatoryRescueReturn = doMandatoryResuce(slot, resize);
 
        // Extract working variables
        bool needsResize = mandatoryRescueReturn.getFirst();
@@ -1266,45 +1211,47 @@ ThreeTuple<bool, int32_t, bool> *Table::fillSlot(Slot slot, bool resize, NewKey
 
        if (needsResize && !resize) {
                // We need to resize but we are not resizing so return false
-               return new ThreeTuple<bool, int32_t, bool>(true, NULL, NULL);
+               return ThreeTuple<bool, int32_t, bool>(true, NULL, NULL);
        }
 
        bool inserted = false;
        if (newKeyEntry != NULL) {
-               newKeyEntry.setSlot(slot);
-               if (slot.hasSpace(newKeyEntry)) {
-
-                       slot.addEntry(newKeyEntry);
+               newKeyEntry->setSlot(slot);
+               if (slot->hasSpace(newKeyEntry)) {
+                       slot->addEntry(newKeyEntry);
                        inserted = true;
                }
        }
 
        // Clear the transactions, aborts and commits that were sent previously
-       transactionPartsSent.clear();
-       pendingSendArbitrationEntriesToDelete.clear();
-
-       for (ArbitrationRound round : pendingSendArbitrationRounds) {
+       transactionPartsSent->clear();
+       pendingSendArbitrationEntriesToDelete->clear();
+       uint size = pendingSendArbitrationRounds->size();
+       for (uint i = 0; i < size; i++) {
+               ArbitrationRound *round = pendingSendArbitrationRounds->get(i);
                bool isFull = false;
-               round.generateParts();
-               Vector<Entry> parts = round.getParts();
+               round->generateParts();
+               Vector<Entry *> *parts = round->getParts();
 
                // Insert pending arbitration data
-               for (Entry arbitrationData : parts) {
+               uint vsize = parts->size();
+               for (uint vi = 0; vi < vsize; vi++) {
+                       Entry *arbitrationData = parts->get(vi);
 
                        // If it is an abort then we need to set some information
-                       if (arbitrationData instanceof Abort) {
-                               ((Abort)arbitrationData).setSequenceNumber(slot.getSequenceNumber());
+                       if (arbitrationData->getType() == TypeAbort) {
+                               ((Abort *)arbitrationData)->setSequenceNumber(slot->getSequenceNumber());
                        }
 
-                       if (!slot.hasSpace(arbitrationData)) {
+                       if (!slot->hasSpace(arbitrationData)) {
                                // No space so cant do anything else with these data entries
                                isFull = true;
                                break;
                        }
 
                        // Add to this current slot and add it to entries to delete
-                       slot.addEntry(arbitrationData);
-                       pendingSendArbitrationEntriesToDelete.add(arbitrationData);
+                       slot->addEntry(arbitrationData);
+                       pendingSendArbitrationEntriesToDelete->add(arbitrationData);
                }
 
                if (isFull) {
@@ -1312,37 +1259,29 @@ ThreeTuple<bool, int32_t, bool> *Table::fillSlot(Slot slot, bool resize, NewKey
                }
        }
 
-       if (pendingTransactionQueue.size() > 0) {
-
-               Transaction transaction = pendingTransactionQueue.get(0);
-
+       if (pendingTransactionQueue->size() > 0) {
+               Transaction *transaction = pendingTransactionQueue->get(0);
                // Set the transaction sequence number if it has yet to be inserted into the block chain
-               // if ((!transaction.didSendAPartToServer() && !transaction.getServerFailure()) || (transaction.getSequenceNumber() == -1)) {
-               //  transaction.setSequenceNumber(slot.getSequenceNumber());
-               // }
-
-               if ((!transaction.didSendAPartToServer()) || (transaction.getSequenceNumber() == -1)) {
-                       transaction.setSequenceNumber(slot.getSequenceNumber());
+               if ((!transaction->didSendAPartToServer()) || (transaction->getSequenceNumber() == -1)) {
+                       transaction->setSequenceNumber(slot->getSequenceNumber());
                }
 
-
                while (true) {
-                       TransactionPart part = transaction.getNextPartToSend();
-
+                       TransactionPart *part = transaction->getNextPartToSend();
                        if (part == NULL) {
                                // Ran out of parts to send for this transaction so move on
                                break;
                        }
 
-                       if (slot.hasSpace(part)) {
-                               slot.addEntry(part);
-                               Vector<int32_t> partsSent = transactionPartsSent.get(transaction);
+                       if (slot->hasSpace(part)) {
+                               slot->addEntry(part);
+                               Vector<int32_t> *partsSent = transactionPartsSent->get(transaction);
                                if (partsSent == NULL) {
                                        partsSent = new Vector<int32_t>();
-                                       transactionPartsSent.put(transaction, partsSent);
+                                       transactionPartsSent->put(transaction, partsSent);
                                }
-                               partsSent.add(part.getPartNumber());
-                               transactionPartsSent.put(transaction, partsSent);
+                               partsSent->add(part->getPartNumber());
+                               transactionPartsSent->put(transaction, partsSent);
                        } else {
                                break;
                        }
@@ -1352,51 +1291,51 @@ ThreeTuple<bool, int32_t, bool> *Table::fillSlot(Slot slot, bool resize, NewKey
        // Fill the remainder of the slot with rescue data
        doOptionalRescue(slot, seenLiveSlot, currentRescueSequenceNumber, resize);
 
-       return new ThreeTuple<bool, int32_t, bool>(false, newSize, inserted);
+       return ThreeTuple<bool, int32_t, bool>(false, newSize, inserted);
 }
 
-void Table::doRejectedMessages(Slot s) {
-       if (!rejectedSlotVector.isEmpty()) {
+void Table::doRejectedMessages(Slot *s) {
+       if (!rejectedSlotVector->isEmpty()) {
                /* TODO: We should avoid generating a rejected message entry if
-                * there is already a sufficient entry in the queue (e.g.,
-                * equalsto value of true and same sequence number).  */
-
-               int64_t old_seqn = rejectedSlotVector.firstElement();
-               if (rejectedSlotVector.size() > REJECTED_THRESHOLD) {
-                       int64_t new_seqn = rejectedSlotVector.lastElement();
-                       RejectedMessage rm = new RejectedMessage(s, s.getSequenceNumber(), localMachineId, old_seqn, new_seqn, false);
-                       s.addEntry(rm);
+                * there is already a sufficient entry in the queue (e->g->,
+                * equalsto value of true and same sequence number)->  */
+
+               int64_t old_seqn = rejectedSlotVector->get(0);
+               if (rejectedSlotVector->size() > Table_REJECTED_THRESHOLD) {
+                       int64_t new_seqn = rejectedSlotVector->lastElement();
+                       RejectedMessage *rm = new RejectedMessage(s, s->getSequenceNumber(), localMachineId, old_seqn, new_seqn, false);
+                       s->addEntry(rm);
                } else {
                        int64_t prev_seqn = -1;
                        int i = 0;
                        /* Go through list of missing messages */
-                       for (; i < rejectedSlotVector.size(); i++) {
-                               int64_t curr_seqn = rejectedSlotVector.get(i);
-                               Slot s_msg = buffer.getSlot(curr_seqn);
+                       for (; i < rejectedSlotVector->size(); i++) {
+                               int64_t curr_seqn = rejectedSlotVector->get(i);
+                               Slot *s_msg = buffer->getSlot(curr_seqn);
                                if (s_msg != NULL)
                                        break;
                                prev_seqn = curr_seqn;
                        }
                        /* Generate rejected message entry for missing messages */
                        if (prev_seqn != -1) {
-                               RejectedMessage rm = new RejectedMessage(s, s.getSequenceNumber(), localMachineId, old_seqn, prev_seqn, false);
-                               s.addEntry(rm);
+                               RejectedMessage *rm = new RejectedMessage(s, s->getSequenceNumber(), localMachineId, old_seqn, prev_seqn, false);
+                               s->addEntry(rm);
                        }
                        /* Generate rejected message entries for present messages */
-                       for (; i < rejectedSlotVector.size(); i++) {
-                               int64_t curr_seqn = rejectedSlotVector.get(i);
-                               Slot s_msg = buffer.getSlot(curr_seqn);
-                               int64_t machineid = s_msg.getMachineID();
-                               RejectedMessage rm = new RejectedMessage(s, s.getSequenceNumber(), machineid, curr_seqn, curr_seqn, true);
-                               s.addEntry(rm);
+                       for (; i < rejectedSlotVector->size(); i++) {
+                               int64_t curr_seqn = rejectedSlotVector->get(i);
+                               Slot *s_msg = buffer->getSlot(curr_seqn);
+                               int64_t machineid = s_msg->getMachineID();
+                               RejectedMessage *rm = new RejectedMessage(s, s->getSequenceNumber(), machineid, curr_seqn, curr_seqn, true);
+                               s->addEntry(rm);
                        }
                }
        }
 }
 
-ThreeTuple<bool, bool, Long> Table::doMandatoryResuce(Slot slot, bool resize) {
-       int64_t newestSequenceNumber = buffer.getNewestSeqNum();
-       int64_t oldestSequenceNumber = buffer.getOldestSeqNum();
+ThreeTuple<bool, bool, int64_t> Table::doMandatoryResuce(Slot *slot, bool resize) {
+       int64_t newestSequenceNumber = buffer->getNewestSeqNum();
+       int64_t oldestSequenceNumber = buffer->getOldestSeqNum();
        if (oldestLiveSlotSequenceNumver < oldestSequenceNumber) {
                oldestLiveSlotSequenceNumver = oldestSequenceNumber;
        }
@@ -1404,18 +1343,18 @@ ThreeTuple<bool, bool, Long> Table::doMandatoryResuce(Slot slot, bool resize) {
        int64_t currentSequenceNumber = oldestLiveSlotSequenceNumver;
        bool seenLiveSlot = false;
        int64_t firstIfFull = newestSequenceNumber + 1 - numberOfSlots;         // smallest seq number in the buffer if it is full
-       int64_t threshold = firstIfFull + FREE_SLOTS;           // we want the buffer to be clear of live entries up to this point
+       int64_t threshold = firstIfFull + Table_FREE_SLOTS;             // we want the buffer to be clear of live entries up to this point
 
 
        // Mandatory Rescue
        for (; currentSequenceNumber < threshold; currentSequenceNumber++) {
-               Slot previousSlot = buffer.getSlot(currentSequenceNumber);
+               Slot *previousSlot = buffer->getSlot(currentSequenceNumber);
                // Push slot number forward
                if (!seenLiveSlot) {
                        oldestLiveSlotSequenceNumver = currentSequenceNumber;
                }
 
-               if (!previousSlot.isLive()) {
+               if (!previousSlot->isLive()) {
                        continue;
                }
 
@@ -1423,123 +1362,140 @@ ThreeTuple<bool, bool, Long> Table::doMandatoryResuce(Slot slot, bool resize) {
                seenLiveSlot = true;
 
                // Get all the live entries for a slot
-               Vector<Entry> liveEntries = previousSlot.getLiveEntries(resize);
+               Vector<Entry *> *liveEntries = previousSlot->getLiveEntries(resize);
 
                // Iterate over all the live entries and try to rescue them
-               for (Entry liveEntry : liveEntries) {
-                       if (slot.hasSpace(liveEntry)) {
-
+               uint lESize = liveEntries->size();
+               for (uint i = 0; i < lESize; i++) {
+                       Entry *liveEntry = liveEntries->get(i);
+                       if (slot->hasSpace(liveEntry)) {
                                // Enough space to rescue the entry
-                               slot.addEntry(liveEntry);
+                               slot->addEntry(liveEntry);
                        } else if (currentSequenceNumber == firstIfFull) {
                                //if there's no space but the entry is about to fall off the queue
-                               System.out.println("B");        //?
-                               return new ThreeTuple<bool, bool, Long>(true, seenLiveSlot, currentSequenceNumber);
-
+                               return ThreeTuple<bool, bool, int64_t>(true, seenLiveSlot, currentSequenceNumber);
                        }
                }
        }
 
        // Did not resize
-       return new ThreeTuple<bool, bool, Long>(false, seenLiveSlot, currentSequenceNumber);
+       return ThreeTuple<bool, bool, int64_t>(false, seenLiveSlot, currentSequenceNumber);
 }
 
-void Table::doOptionalRescue(Slot s, bool seenliveslot, int64_t seqn, bool resize) {
+void Table::doOptionalRescue(Slot *s, bool seenliveslot, int64_t seqn, bool resize) {
        /* now go through live entries from least to greatest sequence number until
         * either all live slots added, or the slot doesn't have enough room
         * for SKIP_THRESHOLD consecutive entries*/
        int skipcount = 0;
-       int64_t newestseqnum = buffer.getNewestSeqNum();
-search:
+       int64_t newestseqnum = buffer->getNewestSeqNum();
        for (; seqn <= newestseqnum; seqn++) {
-               Slot prevslot = buffer.getSlot(seqn);
+               Slot *prevslot = buffer->getSlot(seqn);
                //Push slot number forward
                if (!seenliveslot)
                        oldestLiveSlotSequenceNumver = seqn;
 
-               if (!prevslot.isLive())
+               if (!prevslot->isLive())
                        continue;
                seenliveslot = true;
-               Vector<Entry> liveentries = prevslot.getLiveEntries(resize);
-               for (Entry liveentry : liveentries) {
-                       if (s.hasSpace(liveentry))
-                               s.addEntry(liveentry);
+               Vector<Entry *> *liveentries = prevslot->getLiveEntries(resize);
+               uint lESize = liveentries->size();
+               for (uint i = 0; i < lESize; i++) {
+                       Entry *liveentry = liveentries->get(i);
+                       if (s->hasSpace(liveentry))
+                               s->addEntry(liveentry);
                        else {
                                skipcount++;
-                               if (skipcount > SKIP_THRESHOLD)
-                                       break search;
+                               if (skipcount > Table_SKIP_THRESHOLD)
+                                       goto donesearch;
                        }
                }
        }
+donesearch:
+       ;
 }
 
 /**
- * Checks for malicious activity and updates the local copy of the block chain.
+ * Checks for malicious activity and updates the local copy of the block chain->
  */
-void Table::validateAndUpdate(Array<Slot> *newSlots, bool acceptUpdatesToLocal) {
-
-       // The cloud communication layer has checked slot HMACs already before decoding
-       if (newSlots.length == 0) {
+void Table::validateAndUpdate(Array<Slot *> *newSlots, bool acceptUpdatesToLocal) {
+       // The cloud communication layer has checked slot HMACs already
+       // before decoding
+       if (newSlots->length() == 0) {
                return;
        }
 
-       // Make sure all slots are newer than the last largest slot this client has seen
-       int64_t firstSeqNum = newSlots[0].getSequenceNumber();
+       // Make sure all slots are newer than the last largest slot this
+       // client has seen
+       int64_t firstSeqNum = newSlots->get(0)->getSequenceNumber();
        if (firstSeqNum <= sequenceNumber) {
                throw new Error("Server Error: Sent older slots!");
        }
 
-       // Create an object that can access both new slots and slots in our local chain
-       // without committing slots to our local chain
-       SlotIndexer indexer = new SlotIndexer(newSlots, buffer);
+       // Create an object that can access both new slots and slots in our
+       // local chain without committing slots to our local chain
+       SlotIndexer *indexer = new SlotIndexer(newSlots, buffer);
 
        // Check that the HMAC chain is not broken
        checkHMACChain(indexer, newSlots);
 
        // Set to keep track of messages from clients
-       Hashset<Long> machineSet = new Hashset<int64_t>(lastMessageTable.keySet());
+       Hashset<int64_t> *machineSet = new Hashset<int64_t>();
+       {
+               SetIterator<int64_t, Pair<int64_t, Liveness *> *> *lmit = getKeyIterator(lastMessageTable);
+               while (lmit->hasNext())
+                       machineSet->add(lmit->next());
+               delete lmit;
+       }
 
        // Process each slots data
-       for (Slot slot : newSlots) {
-               processSlot(indexer, slot, acceptUpdatesToLocal, machineSet);
-
-               updateExpectedSize();
+       {
+               uint numSlots = newSlots->length();
+               for (uint i = 0; i < numSlots; i++) {
+                       Slot *slot = newSlots->get(i);
+                       processSlot(indexer, slot, acceptUpdatesToLocal, machineSet);
+                       updateExpectedSize();
+               }
        }
 
-       // If there is a gap, check to see if the server sent us everything.
+       // If there is a gap, check to see if the server sent us
+       // everything->
        if (firstSeqNum != (sequenceNumber + 1)) {
 
-               // Check the size of the slots that were sent down by the server.
+               // Check the size of the slots that were sent down by the server->
                // Can only check the size if there was a gap
-               checkNumSlots(newSlots.length);
+               checkNumSlots(newSlots->length());
 
-               // Since there was a gap every machine must have pushed a slot or must have
-               // a last message message.  If not then the server is hiding slots
-               if (!machineSet.isEmpty()) {
-                       throw new Error("Missing record for machines: " + machineSet);
+               // Since there was a gap every machine must have pushed a slot or
+               // must have a last message message-> If not then the server is
+               // hiding slots
+               if (!machineSet->isEmpty()) {
+                       throw new Error("Missing record for machines: ");
                }
        }
 
-       // Update the size of our local block chain.
+       // Update the size of our local block chain->
        commitNewMaxSize();
 
-       // Commit new to slots to the local block chain.
-       for (Slot slot : newSlots) {
+       // Commit new to slots to the local block chain->
+       {
+               uint numSlots = newSlots->length();
+               for (uint i = 0; i < numSlots; i++) {
+                       Slot *slot = newSlots->get(i);
 
-               // Insert this slot into our local block chain copy.
-               buffer.putSlot(slot);
+                       // Insert this slot into our local block chain copy->
+                       buffer->putSlot(slot);
 
-               // Keep track of how many slots are currently live (have live data in them).
-               liveSlotCount++;
+                       // Keep track of how many slots are currently live (have live data
+                       // in them)->
+                       liveSlotCount++;
+               }
        }
-
        // Get the sequence number of the latest slot in the system
-       sequenceNumber = newSlots[newSlots.length - 1].getSequenceNumber();
-
+       sequenceNumber = newSlots->get(newSlots->length() - 1)->getSequenceNumber();
        updateLiveStateFromServer();
 
        // No Need to remember after we pulled from the server
-       offlineTransactionsCommittedAndAtServer.clear();
+       offlineTransactionsCommittedAndAtServer->clear();
 
        // This is invalidated now
        hadPartialSendToServer = false;
@@ -1576,23 +1532,13 @@ void Table::updateLiveStateFromLocal() {
 }
 
 void Table::initExpectedSize(int64_t firstSequenceNumber, int64_t numberOfSlots) {
-       // if (didFindTableStatus) {
-       // return;
-       // }
        int64_t prevslots = firstSequenceNumber;
 
-
        if (didFindTableStatus) {
-               // expectedsize = (prevslots < ((int64_t) numberOfSlots)) ? (int) prevslots : expectedsize;
-               // System.out.println("Here2: " + expectedsize + "    " + numberOfSlots + "   " + prevslots);
-
        } else {
                expectedsize = (prevslots < ((int64_t) numberOfSlots)) ? (int) prevslots : numberOfSlots;
-               // System.out.println("Here: " + expectedsize);
        }
 
-       // System.out.println(numberOfSlots);
-
        didFindTableStatus = true;
        currMaxSize = numberOfSlots;
 }
@@ -1607,115 +1553,124 @@ void Table::updateExpectedSize() {
 
 
 /**
- * Check the size of the block chain to make sure there are enough slots sent back by the server.
- * This is only called when we have a gap between the slots that we have locally and the slots
- * sent by the server therefore in the slots sent by the server there will be at least 1 Table
- * status message
+ * Check the size of the block chain to make sure there are enough
+ * slots sent back by the server-> This is only called when we have a
+ * gap between the slots that we have locally and the slots sent by
+ * the server therefore in the slots sent by the server there will be
+ * at least 1 Table status message
  */
 void Table::checkNumSlots(int numberOfSlots) {
        if (numberOfSlots != expectedsize) {
-               throw new Error("Server Error: Server did not send all slots.  Expected: " + expectedsize + " Received:" + numberOfSlots);
+               throw new Error("Server Error: Server did not send all slots->  Expected: ");
        }
 }
 
-void Table::updateCurrMaxSize(int newmaxsize) {
-       currMaxSize = newmaxsize;
-}
-
-
 /**
- * Update the size of of the local buffer if it is needed.
+ * Update the size of of the local buffer if it is needed->
  */
 void Table::commitNewMaxSize() {
        didFindTableStatus = false;
 
        // Resize the local slot buffer
        if (numberOfSlots != currMaxSize) {
-               buffer.resize((int)currMaxSize);
+               buffer->resize((int32_t)currMaxSize);
        }
 
        // Change the number of local slots to the new size
-       numberOfSlots = (int)currMaxSize;
+       numberOfSlots = (int32_t)currMaxSize;
 
-
-       // Recalculate the resize threshold since the size of the local buffer has changed
+       // Recalculate the resize threshold since the size of the local
+       // buffer has changed
        setResizeThreshold();
 }
 
 /**
- * Process the new transaction parts from this latest round of slots received from the server
+ * Process the new transaction parts from this latest round of slots
+ * received from the server
  */
 void Table::processNewTransactionParts() {
 
-       if (newTransactionParts.size() == 0) {
+       if (newTransactionParts->size() == 0) {
                // Nothing new to process
                return;
        }
 
-       // Iterate through all the machine Ids that we received new parts for
-       for (Long machineId : newTransactionParts.keySet()) {
-               Hashtable<Pair<int64_t int32_t>, TransactionPart> parts = newTransactionParts.get(machineId);
+       // Iterate through all the machine Ids that we received new parts
+       // for
+       SetIterator<int64_t, Hashtable<Pair<int64_t, int32_t> *, TransactionPart *, uintptr_t, 0, pairHashFunction, pairEquals> *> *tpit = getKeyIterator(newTransactionParts);
+       while (tpit->hasNext()) {
+               int64_t machineId = tpit->next();
+               Hashtable<Pair<int64_t, int32_t> *, TransactionPart *, uintptr_t, 0, pairHashFunction, pairEquals> *parts = newTransactionParts->get(machineId);
 
+               SetIterator<Pair<int64_t, int32_t> *, TransactionPart *, uintptr_t, 0, pairHashFunction, pairEquals> *ptit = getKeyIterator(parts);
                // Iterate through all the parts for that machine Id
-               for (Pair<int64_t int32_t> partId : parts.keySet()) {
-                       TransactionPart part = parts.get(partId);
-
-                       Long lastTransactionNumber = lastArbitratedTransactionNumberByArbitratorTable.get(part.getArbitratorId());
-                       if ((lastTransactionNumber != NULL) && (lastTransactionNumber >= part.getSequenceNumber())) {
-                               // Set dead the transaction part
-                               part.setDead();
-                               continue;
+               while (ptit->hasNext()) {
+                       Pair<int64_t, int32_t> *partId = ptit->next();
+                       TransactionPart *part = parts->get(partId);
+
+                       if (lastArbitratedTransactionNumberByArbitratorTable->contains(part->getArbitratorId())) {
+                               int64_t lastTransactionNumber = lastArbitratedTransactionNumberByArbitratorTable->get(part->getArbitratorId());
+                               if (lastTransactionNumber >= part->getSequenceNumber()) {
+                                       // Set dead the transaction part
+                                       part->setDead();
+                                       continue;
+                               }
                        }
 
                        // Get the transaction object for that sequence number
-                       Transaction transaction = liveTransactionBySequenceNumberTable.get(part.getSequenceNumber());
+                       Transaction *transaction = liveTransactionBySequenceNumberTable->get(part->getSequenceNumber());
 
                        if (transaction == NULL) {
                                // This is a new transaction that we dont have so make a new one
                                transaction = new Transaction();
 
                                // Insert this new transaction into the live tables
-                               liveTransactionBySequenceNumberTable.put(part.getSequenceNumber(), transaction);
-                               liveTransactionByTransactionIdTable.put(part.getTransactionId(), transaction);
+                               liveTransactionBySequenceNumberTable->put(part->getSequenceNumber(), transaction);
+                               liveTransactionByTransactionIdTable->put(new Pair<int64_t, int64_t>(part->getTransactionId()), transaction);
                        }
 
                        // Add that part to the transaction
-                       transaction.addPartDecode(part);
+                       transaction->addPartDecode(part);
                }
+               delete ptit;
        }
-
-       // Clear all the new transaction parts in preparation for the next time the server sends slots
-       newTransactionParts.clear();
+       delete tpit;
+       // Clear all the new transaction parts in preparation for the next
+       // time the server sends slots
+       newTransactionParts->clear();
 }
 
-
-int64_t lastSeqNumArbOn = 0;
-
 void Table::arbitrateFromServer() {
 
-       if (liveTransactionBySequenceNumberTable.size() == 0) {
+       if (liveTransactionBySequenceNumberTable->size() == 0) {
                // Nothing to arbitrate on so move on
                return;
        }
 
        // Get the transaction sequence numbers and sort from oldest to newest
-       Vector<Long> transactionSequenceNumbers = new Vector<Long>(liveTransactionBySequenceNumberTable.keySet());
-       Collections.sort(transactionSequenceNumbers);
+       Vector<int64_t> *transactionSequenceNumbers = new Vector<int64_t>();
+       {
+               SetIterator<int64_t, Transaction *> *trit = getKeyIterator(liveTransactionBySequenceNumberTable);
+               while (trit->hasNext())
+                       transactionSequenceNumbers->add(trit->next());
+               delete trit;
+       }
+       qsort(transactionSequenceNumbers->expose(), transactionSequenceNumbers->size(), sizeof(int64_t), compareInt64);
 
        // Collection of key value pairs that are
-       Hashtable<IoTString, KeyValue> speculativeTableTmp = new Hashtable<IoTString, KeyValue>();
+       Hashtable<IoTString *, KeyValue *> *speculativeTableTmp = new Hashtable<IoTString *, KeyValue *>();
 
        // The last transaction arbitrated on
        int64_t lastTransactionCommitted = -1;
-       Set<Abort> generatedAborts = new Hashset<Abort>();
-
-       for (Long transactionSequenceNumber : transactionSequenceNumbers) {
-               Transaction transaction = liveTransactionBySequenceNumberTable.get(transactionSequenceNumber);
-
-
-
-               // Check if this machine arbitrates for this transaction if not then we cant arbitrate this transaction
-               if (transaction.getArbitrator() != localMachineId) {
+       Hashset<Abort *> *generatedAborts = new Hashset<Abort *>();
+       uint tsnSize = transactionSequenceNumbers->size();
+       for (uint i = 0; i < tsnSize; i++) {
+               int64_t transactionSequenceNumber = transactionSequenceNumbers->get(i);
+               Transaction *transaction = liveTransactionBySequenceNumberTable->get(transactionSequenceNumber);
+
+               // Check if this machine arbitrates for this transaction if not
+               // then we cant arbitrate this transaction
+               if (transaction->getArbitrator() != localMachineId) {
                        continue;
                }
 
@@ -1723,13 +1678,13 @@ void Table::arbitrateFromServer() {
                        continue;
                }
 
-               if (offlineTransactionsCommittedAndAtServer.contains(transaction.getId())) {
+               if (offlineTransactionsCommittedAndAtServer->contains(transaction->getId())) {
                        // We have seen this already locally so dont commit again
                        continue;
                }
 
 
-               if (!transaction.isComplete()) {
+               if (!transaction->isComplete()) {
                        // Will arbitrate in incorrect order if we continue so just break
                        // Most likely this
                        break;
@@ -1737,81 +1692,89 @@ void Table::arbitrateFromServer() {
 
 
                // update the largest transaction seen by arbitrator from server
-               if (lastTransactionSeenFromMachineFromServer.get(transaction.getMachineId()) == NULL) {
-                       lastTransactionSeenFromMachineFromServer.put(transaction.getMachineId(), transaction.getClientLocalSequenceNumber());
+               if (!lastTransactionSeenFromMachineFromServer->contains(transaction->getMachineId())) {
+                       lastTransactionSeenFromMachineFromServer->put(transaction->getMachineId(), transaction->getClientLocalSequenceNumber());
                } else {
-                       Long lastTransactionSeenFromMachine = lastTransactionSeenFromMachineFromServer.get(transaction.getMachineId());
-                       if (transaction.getClientLocalSequenceNumber() > lastTransactionSeenFromMachine) {
-                               lastTransactionSeenFromMachineFromServer.put(transaction.getMachineId(), transaction.getClientLocalSequenceNumber());
+                       int64_t lastTransactionSeenFromMachine = lastTransactionSeenFromMachineFromServer->get(transaction->getMachineId());
+                       if (transaction->getClientLocalSequenceNumber() > lastTransactionSeenFromMachine) {
+                               lastTransactionSeenFromMachineFromServer->put(transaction->getMachineId(), transaction->getClientLocalSequenceNumber());
                        }
                }
 
-               if (transaction.evaluateGuard(committedKeyValueTable, speculativeTableTmp, NULL)) {
+               if (transaction->evaluateGuard(committedKeyValueTable, speculativeTableTmp, NULL)) {
                        // Guard evaluated as true
 
                        // Update the local changes so we can make the commit
-                       for (KeyValue kv : transaction.getKeyValueUpdateSet()) {
-                               speculativeTableTmp.put(kv.getKey(), kv);
+                       SetIterator<KeyValue *, KeyValue *> *kvit = transaction->getKeyValueUpdateSet()->iterator();
+                       while (kvit->hasNext()) {
+                               KeyValue *kv = kvit->next();
+                               speculativeTableTmp->put(kv->getKey(), kv);
                        }
+                       delete kvit;
 
                        // Update what the last transaction committed was for use in batch commit
                        lastTransactionCommitted = transactionSequenceNumber;
                } else {
                        // Guard evaluated was false so create abort
-
                        // Create the abort
-                       Abort newAbort = new Abort(NULL,
-                                                                                                                                transaction.getClientLocalSequenceNumber(),
-                                                                                                                                transaction.getSequenceNumber(),
-                                                                                                                                transaction.getMachineId(),
-                                                                                                                                transaction.getArbitrator(),
-                                                                                                                                localArbitrationSequenceNumber);
+                       Abort *newAbort = new Abort(NULL,
+                                                                                                                                       transaction->getClientLocalSequenceNumber(),
+                                                                                                                                       transaction->getSequenceNumber(),
+                                                                                                                                       transaction->getMachineId(),
+                                                                                                                                       transaction->getArbitrator(),
+                                                                                                                                       localArbitrationSequenceNumber);
                        localArbitrationSequenceNumber++;
-
-                       generatedAborts.add(newAbort);
+                       generatedAborts->add(newAbort);
 
                        // Insert the abort so we can process
                        processEntry(newAbort);
                }
 
                lastSeqNumArbOn = transactionSequenceNumber;
-
-               // liveTransactionBySequenceNumberTable.remove(transactionSequenceNumber);
        }
 
-       Commit newCommit = NULL;
+       Commit *newCommit = NULL;
 
        // If there is something to commit
-       if (speculativeTableTmp.size() != 0) {
-
+       if (speculativeTableTmp->size() != 0) {
                // Create the commit and increment the commit sequence number
                newCommit = new Commit(localArbitrationSequenceNumber, localMachineId, lastTransactionCommitted);
                localArbitrationSequenceNumber++;
 
                // Add all the new keys to the commit
-               for (KeyValue kv : speculativeTableTmp.values()) {
-                       newCommit.addKV(kv);
+               SetIterator<IoTString *, KeyValue *> *spit = getKeyIterator(speculativeTableTmp);
+               while (spit->hasNext()) {
+                       IoTString *string = spit->next();
+                       KeyValue *kv = speculativeTableTmp->get(string);
+                       newCommit->addKV(kv);
                }
+               delete spit;
 
                // create the commit parts
-               newCommit.createCommitParts();
-
-               // Append all the commit parts to the end of the pending queue waiting for sending to the server
+               newCommit->createCommitParts();
 
+               // Append all the commit parts to the end of the pending queue
+               // waiting for sending to the server
                // Insert the commit so we can process it
-               for (CommitPart commitPart : newCommit.getParts().values()) {
+               Vector<CommitPart *> *parts = newCommit->getParts();
+               uint partsSize = parts->size();
+               for (uint i = 0; i < partsSize; i++) {
+                       CommitPart *commitPart = parts->get(i);
                        processEntry(commitPart);
                }
        }
 
-       if ((newCommit != NULL) || (generatedAborts.size() > 0)) {
-               ArbitrationRound arbitrationRound = new ArbitrationRound(newCommit, generatedAborts);
-               pendingSendArbitrationRounds.add(arbitrationRound);
+       if ((newCommit != NULL) || (generatedAborts->size() > 0)) {
+               ArbitrationRound *arbitrationRound = new ArbitrationRound(newCommit, generatedAborts);
+               pendingSendArbitrationRounds->add(arbitrationRound);
 
                if (compactArbitrationData()) {
-                       ArbitrationRound newArbitrationRound = pendingSendArbitrationRounds.get(pendingSendArbitrationRounds.size() - 1);
-                       if (newArbitrationRound.getCommit() != NULL) {
-                               for (CommitPart commitPart : newArbitrationRound.getCommit().getParts().values()) {
+                       ArbitrationRound *newArbitrationRound = pendingSendArbitrationRounds->get(pendingSendArbitrationRounds->size() - 1);
+                       if (newArbitrationRound->getCommit() != NULL) {
+                               Vector<CommitPart *> *parts = newArbitrationRound->getCommit()->getParts();
+                               uint partsSize = parts->size();
+                               for (uint i = 0; i < partsSize; i++) {
+                                       CommitPart *commitPart = parts->get(i);
                                        processEntry(commitPart);
                                }
                        }
@@ -1819,170 +1782,180 @@ void Table::arbitrateFromServer() {
        }
 }
 
-Pair<bool, bool> Table::arbitrateOnLocalTransaction(Transaction transaction) {
+Pair<bool, bool> Table::arbitrateOnLocalTransaction(Transaction *transaction) {
 
-       // Check if this machine arbitrates for this transaction if not then we cant arbitrate this transaction
-       if (transaction.getArbitrator() != localMachineId) {
-               return new Pair<bool, bool>(false, false);
+       // Check if this machine arbitrates for this transaction if not then
+       // we cant arbitrate this transaction
+       if (transaction->getArbitrator() != localMachineId) {
+               return Pair<bool, bool>(false, false);
        }
 
-       if (!transaction.isComplete()) {
+       if (!transaction->isComplete()) {
                // Will arbitrate in incorrect order if we continue so just break
                // Most likely this
-               return new Pair<bool, bool>(false, false);
+               return Pair<bool, bool>(false, false);
        }
 
-       if (transaction.getMachineId() != localMachineId) {
+       if (transaction->getMachineId() != localMachineId) {
                // dont do this check for local transactions
-               if (lastTransactionSeenFromMachineFromServer.get(transaction.getMachineId()) != NULL) {
-                       if (lastTransactionSeenFromMachineFromServer.get(transaction.getMachineId()) > transaction.getClientLocalSequenceNumber()) {
+               if (lastTransactionSeenFromMachineFromServer->contains(transaction->getMachineId())) {
+                       if (lastTransactionSeenFromMachineFromServer->get(transaction->getMachineId()) > transaction->getClientLocalSequenceNumber()) {
                                // We've have already seen this from the server
-                               return new Pair<bool, bool>(false, false);
+                               return Pair<bool, bool>(false, false);
                        }
                }
        }
 
-       if (transaction.evaluateGuard(committedKeyValueTable, NULL, NULL)) {
-               // Guard evaluated as true
-
-               // Create the commit and increment the commit sequence number
-               Commit newCommit = new Commit(localArbitrationSequenceNumber, localMachineId, -1);
+       if (transaction->evaluateGuard(committedKeyValueTable, NULL, NULL)) {
+               // Guard evaluated as true Create the commit and increment the
+               // commit sequence number
+               Commit *newCommit = new Commit(localArbitrationSequenceNumber, localMachineId, -1);
                localArbitrationSequenceNumber++;
 
                // Update the local changes so we can make the commit
-               for (KeyValue kv : transaction.getKeyValueUpdateSet()) {
-                       newCommit.addKV(kv);
+               SetIterator<KeyValue *, KeyValue *> *kvit = transaction->getKeyValueUpdateSet()->iterator();
+               while (kvit->hasNext()) {
+                       KeyValue *kv = kvit->next();
+                       newCommit->addKV(kv);
                }
+               delete kvit;
 
                // create the commit parts
-               newCommit.createCommitParts();
+               newCommit->createCommitParts();
 
-               // Append all the commit parts to the end of the pending queue waiting for sending to the server
+               // Append all the commit parts to the end of the pending queue
+               // waiting for sending to the server
                ArbitrationRound *arbitrationRound = new ArbitrationRound(newCommit, new Hashset<Abort *>());
-               pendingSendArbitrationRounds.add(arbitrationRound);
+               pendingSendArbitrationRounds->add(arbitrationRound);
 
                if (compactArbitrationData()) {
-                       ArbitrationRound newArbitrationRound = pendingSendArbitrationRounds.get(pendingSendArbitrationRounds.size() - 1);
-                       for (CommitPart commitPart : newArbitrationRound.getCommit().getParts().values()) {
+                       ArbitrationRound *newArbitrationRound = pendingSendArbitrationRounds->get(pendingSendArbitrationRounds->size() - 1);
+                       Vector<CommitPart *> *parts = newArbitrationRound->getCommit()->getParts();
+                       uint partsSize = parts->size();
+                       for (uint i = 0; i < partsSize; i++) {
+                               CommitPart *commitPart = parts->get(i);
                                processEntry(commitPart);
                        }
                } else {
                        // Insert the commit so we can process it
-                       for (CommitPart commitPart : newCommit.getParts().values()) {
+                       Vector<CommitPart *> *parts = newCommit->getParts();
+                       uint partsSize = parts->size();
+                       for (uint i = 0; i < partsSize; i++) {
+                               CommitPart *commitPart = parts->get(i);
                                processEntry(commitPart);
                        }
                }
 
-               if (transaction.getMachineId() == localMachineId) {
-                       TransactionStatus status = transaction.getTransactionStatus();
+               if (transaction->getMachineId() == localMachineId) {
+                       TransactionStatus *status = transaction->getTransactionStatus();
                        if (status != NULL) {
-                               status.setStatus(TransactionStatus.StatusCommitted);
+                               status->setStatus(TransactionStatus_StatusCommitted);
                        }
                }
 
                updateLiveStateFromLocal();
-               return new Pair<bool, bool>(true, true);
+               return Pair<bool, bool>(true, true);
        } else {
-
-               if (transaction.getMachineId() == localMachineId) {
+               if (transaction->getMachineId() == localMachineId) {
                        // For locally created messages update the status
-
                        // Guard evaluated was false so create abort
-                       TransactionStatus status = transaction.getTransactionStatus();
+                       TransactionStatus *status = transaction->getTransactionStatus();
                        if (status != NULL) {
-                               status.setStatus(TransactionStatus.StatusAborted);
+                               status->setStatus(TransactionStatus_StatusAborted);
                        }
                } else {
-                       Hashset<Abort *> addAbortSet = new Hashset<Abort * >();
-
+                       Hashset<Abort *> *addAbortSet = new Hashset<Abort * >();
 
                        // Create the abort
-                       Abort newAbort = new Abort(NULL,
-                                                                                                                                transaction.getClientLocalSequenceNumber(),
-                                                                                                                                -1,
-                                                                                                                                transaction.getMachineId(),
-                                                                                                                                transaction.getArbitrator(),
-                                                                                                                                localArbitrationSequenceNumber);
+                       Abort *newAbort = new Abort(NULL,
+                                                                                                                                       transaction->getClientLocalSequenceNumber(),
+                                                                                                                                       -1,
+                                                                                                                                       transaction->getMachineId(),
+                                                                                                                                       transaction->getArbitrator(),
+                                                                                                                                       localArbitrationSequenceNumber);
                        localArbitrationSequenceNumber++;
+                       addAbortSet->add(newAbort);
 
-                       addAbortSet.add(newAbort);
-
-
-                       // Append all the commit parts to the end of the pending queue waiting for sending to the server
-                       ArbitrationRound arbitrationRound = new ArbitrationRound(NULL, addAbortSet);
-                       pendingSendArbitrationRounds.add(arbitrationRound);
+                       // Append all the commit parts to the end of the pending queue
+                       // waiting for sending to the server
+                       ArbitrationRound *arbitrationRound = new ArbitrationRound(NULL, addAbortSet);
+                       pendingSendArbitrationRounds->add(arbitrationRound);
 
                        if (compactArbitrationData()) {
-                               ArbitrationRound newArbitrationRound = pendingSendArbitrationRounds.get(pendingSendArbitrationRounds.size() - 1);
-                               for (CommitPart commitPart : newArbitrationRound.getCommit().getParts().values()) {
+                               ArbitrationRound *newArbitrationRound = pendingSendArbitrationRounds->get(pendingSendArbitrationRounds->size() - 1);
+
+                               Vector<CommitPart *> *parts = newArbitrationRound->getCommit()->getParts();
+                               uint partsSize = parts->size();
+                               for (uint i = 0; i < partsSize; i++) {
+                                       CommitPart *commitPart = parts->get(i);
                                        processEntry(commitPart);
                                }
                        }
                }
 
                updateLiveStateFromLocal();
-               return new Pair<bool, bool>(true, false);
+               return Pair<bool, bool>(true, false);
        }
 }
 
 /**
- * Compacts the arbitration data my merging commits and aggregating aborts so that a single large push of commits can be done instead of many small updates
+ * Compacts the arbitration data my merging commits and aggregating
+ * aborts so that a single large push of commits can be done instead
+ * of many small updates
  */
 bool Table::compactArbitrationData() {
-
-       if (pendingSendArbitrationRounds.size() < 2) {
+       if (pendingSendArbitrationRounds->size() < 2) {
                // Nothing to compact so do nothing
                return false;
        }
 
-       ArbitrationRound lastRound = pendingSendArbitrationRounds.get(pendingSendArbitrationRounds.size() - 1);
-       if (lastRound.didSendPart()) {
+       ArbitrationRound *lastRound = pendingSendArbitrationRounds->get(pendingSendArbitrationRounds->size() - 1);
+       if (lastRound->getDidSendPart()) {
                return false;
        }
 
-       bool hadCommit = (lastRound.getCommit() == NULL);
+       bool hadCommit = (lastRound->getCommit() == NULL);
        bool gotNewCommit = false;
 
        int numberToDelete = 1;
-       while (numberToDelete < pendingSendArbitrationRounds.size()) {
-               ArbitrationRound round = pendingSendArbitrationRounds.get(pendingSendArbitrationRounds.size() - numberToDelete - 1);
+       while (numberToDelete < pendingSendArbitrationRounds->size()) {
+               ArbitrationRound *round = pendingSendArbitrationRounds->get(pendingSendArbitrationRounds->size() - numberToDelete - 1);
 
-               if (round.isFull() || round.didSendPart()) {
-                       // Stop since there is a part that cannot be compacted and we need to compact in order
+               if (round->isFull() || round->getDidSendPart()) {
+                       // Stop since there is a part that cannot be compacted and we
+                       // need to compact in order
                        break;
                }
 
-               if (round.getCommit() == NULL) {
-
+               if (round->getCommit() == NULL) {
                        // Try compacting aborts only
-                       int newSize = round.getCurrentSize() + lastRound.getAbortsCount();
-                       if (newSize > ArbitrationRound.MAX_PARTS) {
+                       int newSize = round->getCurrentSize() + lastRound->getAbortsCount();
+                       if (newSize > ArbitrationRound_MAX_PARTS) {
                                // Cant compact since it would be too large
                                break;
                        }
-                       lastRound.addAborts(round.getAborts());
+                       lastRound->addAborts(round->getAborts());
                } else {
-
                        // Create a new larger commit
-                       Commit newCommit = Commit.merge(lastRound.getCommit(), round.getCommit(), localArbitrationSequenceNumber);
+                       Commit *newCommit = Commit_merge(lastRound->getCommit(), round->getCommit(), localArbitrationSequenceNumber);
                        localArbitrationSequenceNumber++;
 
                        // Create the commit parts so that we can count them
-                       newCommit.createCommitParts();
+                       newCommit->createCommitParts();
 
                        // Calculate the new size of the parts
-                       int newSize = newCommit.getNumberOfParts();
-                       newSize += lastRound.getAbortsCount();
-                       newSize += round.getAbortsCount();
+                       int newSize = newCommit->getNumberOfParts();
+                       newSize += lastRound->getAbortsCount();
+                       newSize += round->getAbortsCount();
 
-                       if (newSize > ArbitrationRound.MAX_PARTS) {
+                       if (newSize > ArbitrationRound_MAX_PARTS) {
                                // Cant compact since it would be too large
                                break;
                        }
 
                        // Set the new compacted part
-                       lastRound.setCommit(newCommit);
-                       lastRound.addAborts(round.getAborts());
+                       lastRound->setCommit(newCommit);
+                       lastRound->addAborts(round->getAborts());
                        gotNewCommit = true;
                }
 
@@ -1991,18 +1964,17 @@ bool Table::compactArbitrationData() {
 
        if (numberToDelete != 1) {
                // If there is a compaction
-
                // Delete the previous pieces that are now in the new compacted piece
-               if (numberToDelete == pendingSendArbitrationRounds.size()) {
-                       pendingSendArbitrationRounds.clear();
+               if (numberToDelete == pendingSendArbitrationRounds->size()) {
+                       pendingSendArbitrationRounds->clear();
                } else {
                        for (int i = 0; i < numberToDelete; i++) {
-                               pendingSendArbitrationRounds.remove(pendingSendArbitrationRounds.size() - 1);
+                               pendingSendArbitrationRounds->removeIndex(pendingSendArbitrationRounds->size() - 1);
                        }
                }
 
                // Add the new compacted into the pending to send list
-               pendingSendArbitrationRounds.add(lastRound);
+               pendingSendArbitrationRounds->add(lastRound);
 
                // Should reinsert into the commit processor
                if (hadCommit && gotNewCommit) {
@@ -2012,216 +1984,266 @@ bool Table::compactArbitrationData() {
 
        return false;
 }
-// bool compactArbitrationData() {
-//  return false;
-// }
 
 /**
- * Update all the commits and the committed tables, sets dead the dead transactions
+ * Update all the commits and the committed tables, sets dead the dead
+ * transactions
  */
 bool Table::updateCommittedTable() {
 
-       if (newCommitParts.size() == 0) {
+       if (newCommitParts->size() == 0) {
                // Nothing new to process
                return false;
        }
 
        // Iterate through all the machine Ids that we received new parts for
-       for (Long machineId : newCommitParts.keySet()) {
-               Hashtable<Pair<int64_t int32_t>, CommitPart> parts = newCommitParts.get(machineId);
+       SetIterator<int64_t, Hashtable<Pair<int64_t, int32_t> *, CommitPart *, uintptr_t, 0, pairHashFunction, pairEquals> *> *partsit = getKeyIterator(newCommitParts);
+       while (partsit->hasNext()) {
+               int64_t machineId = partsit->next();
+               Hashtable<Pair<int64_t, int32_t> *, CommitPart *, uintptr_t, 0, pairHashFunction, pairEquals> *parts = newCommitParts->get(machineId);
 
                // Iterate through all the parts for that machine Id
-               for (Pair<int64_t int32_t> partId : parts.keySet()) {
-                       CommitPart part = parts.get(partId);
+               SetIterator<Pair<int64_t, int32_t> *, CommitPart *, uintptr_t, 0, pairHashFunction, pairEquals> *pairit = getKeyIterator(parts);
+               while (pairit->hasNext()) {
+                       Pair<int64_t, int32_t> *partId = pairit->next();
+                       CommitPart *part = parts->get(partId);
 
                        // Get the transaction object for that sequence number
-                       Hashtable<int64_t Commit> commitForClientTable = liveCommitsTable.get(part.getMachineId());
+                       Hashtable<int64_t, Commit *> *commitForClientTable = liveCommitsTable->get(part->getMachineId());
 
                        if (commitForClientTable == NULL) {
                                // This is the first commit from this device
-                               commitForClientTable = new Hashtable<int64_t Commit>();
-                               liveCommitsTable.put(part.getMachineId(), commitForClientTable);
+                               commitForClientTable = new Hashtable<int64_t, Commit *>();
+                               liveCommitsTable->put(part->getMachineId(), commitForClientTable);
                        }
 
-                       Commit commit = commitForClientTable.get(part.getSequenceNumber());
+                       Commit *commit = commitForClientTable->get(part->getSequenceNumber());
 
                        if (commit == NULL) {
                                // This is a new commit that we dont have so make a new one
                                commit = new Commit();
 
                                // Insert this new commit into the live tables
-                               commitForClientTable.put(part.getSequenceNumber(), commit);
+                               commitForClientTable->put(part->getSequenceNumber(), commit);
                        }
 
                        // Add that part to the commit
-                       commit.addPartDecode(part);
+                       commit->addPartDecode(part);
                }
+               delete pairit;
        }
+       delete partsit;
 
-       // Clear all the new commits parts in preparation for the next time the server sends slots
-       newCommitParts.clear();
+       // Clear all the new commits parts in preparation for the next time
+       // the server sends slots
+       newCommitParts->clear();
 
        // If we process a new commit keep track of it for future use
        bool didProcessANewCommit = false;
 
        // Process the commits one by one
-       for (Long arbitratorId : liveCommitsTable.keySet()) {
+       SetIterator<int64_t, Hashtable<int64_t, Commit *> *> *liveit = getKeyIterator(liveCommitsTable);
+       while (liveit->hasNext()) {
+               int64_t arbitratorId = liveit->next();
 
                // Get all the commits for a specific arbitrator
-               Hashtable<int64_t Commit> commitForClientTable = liveCommitsTable.get(arbitratorId);
+               Hashtable<int64_t, Commit *> *commitForClientTable = liveCommitsTable->get(arbitratorId);
 
                // Sort the commits in order
-               Vector<Long> commitSequenceNumbers = new Vector<Long>(commitForClientTable.keySet());
-               Collections.sort(commitSequenceNumbers);
+               Vector<int64_t> *commitSequenceNumbers = new Vector<int64_t>();
+               {
+                       SetIterator<int64_t, Commit *> *clientit = getKeyIterator(commitForClientTable);
+                       while (clientit->hasNext())
+                               commitSequenceNumbers->add(clientit->next());
+                       delete clientit;
+               }
+
+               qsort(commitSequenceNumbers->expose(), commitSequenceNumbers->size(), sizeof(int64_t), compareInt64);
 
                // Get the last commit seen from this arbitrator
                int64_t lastCommitSeenSequenceNumber = -1;
-               if (lastCommitSeenSequenceNumberByArbitratorTable.get(arbitratorId) != NULL) {
-                       lastCommitSeenSequenceNumber = lastCommitSeenSequenceNumberByArbitratorTable.get(arbitratorId);
+               if (lastCommitSeenSequenceNumberByArbitratorTable->contains(arbitratorId)) {
+                       lastCommitSeenSequenceNumber = lastCommitSeenSequenceNumberByArbitratorTable->get(arbitratorId);
                }
 
                // Go through each new commit one by one
-               for (int i = 0; i < commitSequenceNumbers.size(); i++) {
-                       Long commitSequenceNumber = commitSequenceNumbers.get(i);
-                       Commit commit = commitForClientTable.get(commitSequenceNumber);
+               for (int i = 0; i < commitSequenceNumbers->size(); i++) {
+                       int64_t commitSequenceNumber = commitSequenceNumbers->get(i);
+                       Commit *commit = commitForClientTable->get(commitSequenceNumber);
 
                        // Special processing if a commit is not complete
-                       if (!commit.isComplete()) {
-                               if (i == (commitSequenceNumbers.size() - 1)) {
-                                       // If there is an incomplete commit and this commit is the latest one seen then this commit cannot be processed and there are no other commits
+                       if (!commit->isComplete()) {
+                               if (i == (commitSequenceNumbers->size() - 1)) {
+                                       // If there is an incomplete commit and this commit is the
+                                       // latest one seen then this commit cannot be processed and
+                                       // there are no other commits
                                        break;
                                } else {
-                                       // This is a commit that was already dead but parts of it are still in the block chain (not flushed out yet).
+                                       // This is a commit that was already dead but parts of it
+                                       // are still in the block chain (not flushed out yet)->
                                        // Delete it and move on
-                                       commit.setDead();
-                                       commitForClientTable.remove(commit.getSequenceNumber());
+                                       commit->setDead();
+                                       commitForClientTable->remove(commit->getSequenceNumber());
                                        continue;
                                }
                        }
 
                        // Update the last transaction that was updated if we can
-                       if (commit.getTransactionSequenceNumber() != -1) {
-                               Long lastTransactionNumber = lastArbitratedTransactionNumberByArbitratorTable.get(commit.getMachineId());
-
-                               // Update the last transaction sequence number that the arbitrator arbitrated on
-                               if ((lastTransactionNumber == NULL) || (lastTransactionNumber < commit.getTransactionSequenceNumber())) {
-                                       lastArbitratedTransactionNumberByArbitratorTable.put(commit.getMachineId(), commit.getTransactionSequenceNumber());
+                       if (commit->getTransactionSequenceNumber() != -1) {
+                               // Update the last transaction sequence number that the arbitrator arbitrated on1
+                               if (!lastArbitratedTransactionNumberByArbitratorTable->contains(commit->getMachineId()) || lastArbitratedTransactionNumberByArbitratorTable->get(commit->getMachineId()) < commit->getTransactionSequenceNumber()) {
+                                       lastArbitratedTransactionNumberByArbitratorTable->put(commit->getMachineId(), commit->getTransactionSequenceNumber());
                                }
                        }
 
                        // Update the last arbitration data that we have seen so far
-                       if (lastArbitrationDataLocalSequenceNumberSeenFromArbitrator.get(commit.getMachineId()) != NULL) {
-
-                               int64_t lastArbitrationSequenceNumber = lastArbitrationDataLocalSequenceNumberSeenFromArbitrator.get(commit.getMachineId());
-                               if (commit.getSequenceNumber() > lastArbitrationSequenceNumber) {
+                       if (lastArbitrationDataLocalSequenceNumberSeenFromArbitrator->contains(commit->getMachineId())) {
+                               int64_t lastArbitrationSequenceNumber = lastArbitrationDataLocalSequenceNumberSeenFromArbitrator->get(commit->getMachineId());
+                               if (commit->getSequenceNumber() > lastArbitrationSequenceNumber) {
                                        // Is larger
-                                       lastArbitrationDataLocalSequenceNumberSeenFromArbitrator.put(commit.getMachineId(), commit.getSequenceNumber());
+                                       lastArbitrationDataLocalSequenceNumberSeenFromArbitrator->put(commit->getMachineId(), commit->getSequenceNumber());
                                }
                        } else {
                                // Never seen any data from this arbitrator so record the first one
-                               lastArbitrationDataLocalSequenceNumberSeenFromArbitrator.put(commit.getMachineId(), commit.getSequenceNumber());
+                               lastArbitrationDataLocalSequenceNumberSeenFromArbitrator->put(commit->getMachineId(), commit->getSequenceNumber());
                        }
 
-                       // We have already seen this commit before so need to do the full processing on this commit
-                       if (commit.getSequenceNumber() <= lastCommitSeenSequenceNumber) {
+                       // We have already seen this commit before so need to do the
+                       // full processing on this commit
+                       if (commit->getSequenceNumber() <= lastCommitSeenSequenceNumber) {
 
                                // Update the last transaction that was updated if we can
-                               if (commit.getTransactionSequenceNumber() != -1) {
-                                       Long lastTransactionNumber = lastArbitratedTransactionNumberByArbitratorTable.get(commit.getMachineId());
-
-                                       // Update the last transaction sequence number that the arbitrator arbitrated on
-                                       if ((lastTransactionNumber == NULL) || (lastTransactionNumber < commit.getTransactionSequenceNumber())) {
-                                               lastArbitratedTransactionNumberByArbitratorTable.put(commit.getMachineId(), commit.getTransactionSequenceNumber());
+                               if (commit->getTransactionSequenceNumber() != -1) {
+                                       int64_t lastTransactionNumber = lastArbitratedTransactionNumberByArbitratorTable->get(commit->getMachineId());
+                                       if (!lastArbitratedTransactionNumberByArbitratorTable->contains(commit->getMachineId()) ||
+                                                       lastArbitratedTransactionNumberByArbitratorTable->get(commit->getMachineId()) < commit->getTransactionSequenceNumber()) {
+                                               lastArbitratedTransactionNumberByArbitratorTable->put(commit->getMachineId(), commit->getTransactionSequenceNumber());
                                        }
                                }
 
                                continue;
                        }
 
-                       // If we got here then this is a brand new commit and needs full processing
-
-                       // Get what commits should be edited, these are the commits that have live values for their keys
+                       // If we got here then this is a brand new commit and needs full
+                       // processing
+                       // Get what commits should be edited, these are the commits that
+                       // have live values for their keys
                        Hashset<Commit *> *commitsToEdit = new Hashset<Commit *>();
-                       for (KeyValue kv : commit.getKeyValueUpdateSet()) {
-                               commitsToEdit.add(liveCommitsByKeyTable.get(kv.getKey()));
+                       {
+                               SetIterator<KeyValue *, KeyValue *, uintptr_t, 0, hashKeyValue, equalsKeyValue> *kvit = commit->getKeyValueUpdateSet()->iterator();
+                               while (kvit->hasNext()) {
+                                       KeyValue *kv = kvit->next();
+                                       Commit *commit = liveCommitsByKeyTable->get(kv->getKey());
+                                       if (commit != NULL)
+                                               commitsToEdit->add(commit);
+                               }
+                               delete kvit;
                        }
-                       commitsToEdit.remove(NULL);             // remove NULL since it could be in this set
 
                        // Update each previous commit that needs to be updated
-                       for (Commit previousCommit : commitsToEdit) {
+                       SetIterator<Commit *, Commit *> *commitit = commitsToEdit->iterator();
+                       while (commitit->hasNext()) {
+                               Commit *previousCommit = commitit->next();
 
                                // Only bother with live commits (TODO: Maybe remove this check)
-                               if (previousCommit.isLive()) {
+                               if (previousCommit->isLive()) {
 
                                        // Update which keys in the old commits are still live
-                                       for (KeyValue kv : commit.getKeyValueUpdateSet()) {
-                                               previousCommit.invalidateKey(kv.getKey());
+                                       {
+                                               SetIterator<KeyValue *, KeyValue *, uintptr_t, 0, hashKeyValue, equalsKeyValue> *kvit = commit->getKeyValueUpdateSet()->iterator();
+                                               while (kvit->hasNext()) {
+                                                       KeyValue *kv = kvit->next();
+                                                       previousCommit->invalidateKey(kv->getKey());
+                                               }
+                                               delete kvit;
                                        }
 
                                        // if the commit is now dead then remove it
-                                       if (!previousCommit.isLive()) {
-                                               commitForClientTable.remove(previousCommit);
+                                       if (!previousCommit->isLive()) {
+                                               commitForClientTable->remove(previousCommit->getSequenceNumber());
                                        }
                                }
                        }
+                       delete commitit;
 
                        // Update the last seen sequence number from this arbitrator
-                       if (lastCommitSeenSequenceNumberByArbitratorTable.get(commit.getMachineId()) != NULL) {
-                               if (commit.getSequenceNumber() > lastCommitSeenSequenceNumberByArbitratorTable.get(commit.getMachineId())) {
-                                       lastCommitSeenSequenceNumberByArbitratorTable.put(commit.getMachineId(), commit.getSequenceNumber());
+                       if (lastCommitSeenSequenceNumberByArbitratorTable->contains(commit->getMachineId())) {
+                               if (commit->getSequenceNumber() > lastCommitSeenSequenceNumberByArbitratorTable->get(commit->getMachineId())) {
+                                       lastCommitSeenSequenceNumberByArbitratorTable->put(commit->getMachineId(), commit->getSequenceNumber());
                                }
                        } else {
-                               lastCommitSeenSequenceNumberByArbitratorTable.put(commit.getMachineId(), commit.getSequenceNumber());
+                               lastCommitSeenSequenceNumberByArbitratorTable->put(commit->getMachineId(), commit->getSequenceNumber());
                        }
 
                        // We processed a new commit that we havent seen before
                        didProcessANewCommit = true;
 
                        // Update the committed table of keys and which commit is using which key
-                       for (KeyValue kv : commit.getKeyValueUpdateSet()) {
-                               committedKeyValueTable.put(kv.getKey(), kv);
-                               liveCommitsByKeyTable.put(kv.getKey(), commit);
+                       {
+                               SetIterator<KeyValue *, KeyValue *, uintptr_t, 0, hashKeyValue, equalsKeyValue> *kvit = commit->getKeyValueUpdateSet()->iterator();
+                               while (kvit->hasNext()) {
+                                       KeyValue *kv = kvit->next();
+                                       committedKeyValueTable->put(kv->getKey(), kv);
+                                       liveCommitsByKeyTable->put(kv->getKey(), commit);
+                               }
+                               delete kvit;
                        }
                }
        }
+       delete liveit;
 
        return didProcessANewCommit;
 }
 
 /**
- * Create the speculative table from transactions that are still live and have come from the cloud
+ * Create the speculative table from transactions that are still live
+ * and have come from the cloud
  */
 bool Table::updateSpeculativeTable(bool didProcessNewCommits) {
-       if (liveTransactionBySequenceNumberTable.keySet().size() == 0) {
+       if (liveTransactionBySequenceNumberTable->size() == 0) {
                // There is nothing to speculate on
                return false;
        }
 
-       // Create a list of the transaction sequence numbers and sort them from oldest to newest
-       Vector<Long> transactionSequenceNumbersSorted = new Vector<Long>(liveTransactionBySequenceNumberTable.keySet());
-       Collections.sort(transactionSequenceNumbersSorted);
+       // Create a list of the transaction sequence numbers and sort them
+       // from oldest to newest
+       Vector<int64_t> *transactionSequenceNumbersSorted = new Vector<int64_t>();
+       {
+               SetIterator<int64_t, Transaction *> *trit = getKeyIterator(liveTransactionBySequenceNumberTable);
+               while (trit->hasNext())
+                       transactionSequenceNumbersSorted->add(trit->next());
+               delete trit;
+       }
+
+       qsort(transactionSequenceNumbersSorted->expose(), transactionSequenceNumbersSorted->size(), sizeof(int64_t), compareInt64);
 
-       bool hasGapInTransactionSequenceNumbers = transactionSequenceNumbersSorted.get(0) != oldestTransactionSequenceNumberSpeculatedOn;
+       bool hasGapInTransactionSequenceNumbers = transactionSequenceNumbersSorted->get(0) != oldestTransactionSequenceNumberSpeculatedOn;
 
 
        if (hasGapInTransactionSequenceNumbers || didProcessNewCommits) {
-               // If there is a gap in the transaction sequence numbers then there was a commit or an abort of a transaction
-               // OR there was a new commit (Could be from offline commit) so a redo the speculation from scratch
+               // If there is a gap in the transaction sequence numbers then
+               // there was a commit or an abort of a transaction OR there was a
+               // new commit (Could be from offline commit) so a redo the
+               // speculation from scratch
 
                // Start from scratch
-               speculatedKeyValueTable.clear();
+               speculatedKeyValueTable->clear();
                lastTransactionSequenceNumberSpeculatedOn = -1;
                oldestTransactionSequenceNumberSpeculatedOn = -1;
-
        }
 
        // Remember the front of the transaction list
-       oldestTransactionSequenceNumberSpeculatedOn = transactionSequenceNumbersSorted.get(0);
+       oldestTransactionSequenceNumberSpeculatedOn = transactionSequenceNumbersSorted->get(0);
 
        // Find where to start arbitration from
-       int startIndex = transactionSequenceNumbersSorted.indexOf(lastTransactionSequenceNumberSpeculatedOn) + 1;
+       int startIndex = 0;
 
-       if (startIndex >= transactionSequenceNumbersSorted.size()) {
+       for (; startIndex < transactionSequenceNumbersSorted->size(); startIndex++)
+               if (transactionSequenceNumbersSorted->get(startIndex) == lastTransactionSequenceNumberSpeculatedOn)
+                       break;
+       startIndex++;
+
+       if (startIndex >= transactionSequenceNumbersSorted->size()) {
                // Make sure we are not out of bounds
                return false;           // did not speculate
        }
@@ -2229,28 +2251,34 @@ bool Table::updateSpeculativeTable(bool didProcessNewCommits) {
        Hashset<int64_t> *incompleteTransactionArbitrator = new Hashset<int64_t>();
        bool didSkip = true;
 
-       for (int i = startIndex; i < transactionSequenceNumbersSorted.size(); i++) {
-               int64_t transactionSequenceNumber = transactionSequenceNumbersSorted.get(i);
-               Transaction transaction = liveTransactionBySequenceNumberTable.get(transactionSequenceNumber);
+       for (int i = startIndex; i < transactionSequenceNumbersSorted->size(); i++) {
+               int64_t transactionSequenceNumber = transactionSequenceNumbersSorted->get(i);
+               Transaction *transaction = liveTransactionBySequenceNumberTable->get(transactionSequenceNumber);
 
-               if (!transaction.isComplete()) {
-                       // If there is an incomplete transaction then there is nothing we can do
-                       // add this transactions arbitrator to the list of arbitrators we should ignore
-                       incompleteTransactionArbitrator.add(transaction.getArbitrator());
+               if (!transaction->isComplete()) {
+                       // If there is an incomplete transaction then there is nothing
+                       // we can do add this transactions arbitrator to the list of
+                       // arbitrators we should ignore
+                       incompleteTransactionArbitrator->add(transaction->getArbitrator());
                        didSkip = true;
                        continue;
                }
 
-               if (incompleteTransactionArbitrator.contains(transaction.getArbitrator())) {
+               if (incompleteTransactionArbitrator->contains(transaction->getArbitrator())) {
                        continue;
                }
 
                lastTransactionSequenceNumberSpeculatedOn = transactionSequenceNumber;
 
-               if (transaction.evaluateGuard(committedKeyValueTable, speculatedKeyValueTable, NULL)) {
+               if (transaction->evaluateGuard(committedKeyValueTable, speculatedKeyValueTable, NULL)) {
                        // Guard evaluated to true so update the speculative table
-                       for (KeyValue kv : transaction.getKeyValueUpdateSet()) {
-                               speculatedKeyValueTable.put(kv.getKey(), kv);
+                       {
+                               SetIterator<KeyValue *, KeyValue *> *kvit = transaction->getKeyValueUpdateSet()->iterator();
+                               while (kvit->hasNext()) {
+                                       KeyValue *kv = kvit->next();
+                                       speculatedKeyValueTable->put(kv->getKey(), kv);
+                               }
+                               delete kvit;
                        }
                }
        }
@@ -2266,125 +2294,134 @@ bool Table::updateSpeculativeTable(bool didProcessNewCommits) {
 }
 
 /**
- * Create the pending transaction speculative table from transactions that are still in the pending transaction buffer
+ * Create the pending transaction speculative table from transactions
+ * that are still in the pending transaction buffer
  */
 void Table::updatePendingTransactionSpeculativeTable(bool didProcessNewCommitsOrSpeculate) {
-       if (pendingTransactionQueue.size() == 0) {
+       if (pendingTransactionQueue->size() == 0) {
                // There is nothing to speculate on
                return;
        }
 
-
-       if (didProcessNewCommitsOrSpeculate || (firstPendingTransaction != pendingTransactionQueue.get(0))) {
+       if (didProcessNewCommitsOrSpeculate || (firstPendingTransaction != pendingTransactionQueue->get(0))) {
                // need to reset on the pending speculation
                lastPendingTransactionSpeculatedOn = NULL;
-               firstPendingTransaction = pendingTransactionQueue.get(0);
-               pendingTransactionSpeculatedKeyValueTable.clear();
+               firstPendingTransaction = pendingTransactionQueue->get(0);
+               pendingTransactionSpeculatedKeyValueTable->clear();
        }
 
        // Find where to start arbitration from
-       int startIndex = pendingTransactionQueue.indexOf(firstPendingTransaction) + 1;
+       int startIndex = 0;
 
-       if (startIndex >= pendingTransactionQueue.size()) {
+       for (; startIndex < pendingTransactionQueue->size(); startIndex++)
+               if (pendingTransactionQueue->get(startIndex) == firstPendingTransaction)
+                       break;
+
+       if (startIndex >= pendingTransactionQueue->size()) {
                // Make sure we are not out of bounds
                return;
        }
 
-       for (int i = startIndex; i < pendingTransactionQueue.size(); i++) {
-               Transaction transaction = pendingTransactionQueue.get(i);
+       for (int i = startIndex; i < pendingTransactionQueue->size(); i++) {
+               Transaction *transaction = pendingTransactionQueue->get(i);
 
                lastPendingTransactionSpeculatedOn = transaction;
 
-               if (transaction.evaluateGuard(committedKeyValueTable, speculatedKeyValueTable, pendingTransactionSpeculatedKeyValueTable)) {
+               if (transaction->evaluateGuard(committedKeyValueTable, speculatedKeyValueTable, pendingTransactionSpeculatedKeyValueTable)) {
                        // Guard evaluated to true so update the speculative table
-                       for (KeyValue kv : transaction.getKeyValueUpdateSet()) {
-                               pendingTransactionSpeculatedKeyValueTable.put(kv.getKey(), kv);
+                       SetIterator<KeyValue *, KeyValue *> *kvit = transaction->getKeyValueUpdateSet()->iterator();
+                       while (kvit->hasNext()) {
+                               KeyValue *kv = kvit->next();
+                               pendingTransactionSpeculatedKeyValueTable->put(kv->getKey(), kv);
                        }
+                       delete kvit;
                }
        }
 }
 
 /**
- * Set dead and remove from the live transaction tables the transactions that are dead
+ * Set dead and remove from the live transaction tables the
+ * transactions that are dead
  */
 void Table::updateLiveTransactionsAndStatus() {
-
        // Go through each of the transactions
-       for (Iterator<Map.Entry<int64_t Transaction> > iter = liveTransactionBySequenceNumberTable.entrySet().iterator(); iter.hasNext();) {
-               Transaction transaction = iter.next().getValue();
-
-               // Check if the transaction is dead
-               Long lastTransactionNumber = lastArbitratedTransactionNumberByArbitratorTable.get(transaction.getArbitrator());
-               if ((lastTransactionNumber != NULL) && (lastTransactionNumber >= transaction.getSequenceNumber())) {
-
-                       // Set dead the transaction
-                       transaction.setDead();
-
-                       // Remove the transaction from the live table
-                       iter.remove();
-                       liveTransactionByTransactionIdTable.remove(transaction.getId());
+       {
+               SetIterator<int64_t, Transaction *> *iter = getKeyIterator(liveTransactionBySequenceNumberTable);
+               while (iter->hasNext()) {
+                       int64_t key = iter->next();
+                       Transaction *transaction = liveTransactionBySequenceNumberTable->get(key);
+
+                       // Check if the transaction is dead
+                       if (lastArbitratedTransactionNumberByArbitratorTable->contains(transaction->getArbitrator()) && lastArbitratedTransactionNumberByArbitratorTable->get(transaction->getArbitrator() >= transaction->getSequenceNumber())) {
+                               // Set dead the transaction
+                               transaction->setDead();
+
+                               // Remove the transaction from the live table
+                               iter->remove();
+                               liveTransactionByTransactionIdTable->remove(transaction->getId());
+                       }
                }
+               delete iter;
        }
 
        // Go through each of the transactions
-       for (Iterator<Map.Entry<int64_t TransactionStatus> > iter = outstandingTransactionStatus.entrySet().iterator(); iter.hasNext();) {
-               TransactionStatus status = iter.next().getValue();
+       {
+               SetIterator<int64_t, TransactionStatus *> *iter = getKeyIterator(outstandingTransactionStatus);
+               while (iter->hasNext()) {
+                       int64_t key = iter->next();
+                       TransactionStatus *status = outstandingTransactionStatus->get(key);
 
-               // Check if the transaction is dead
-               Long lastTransactionNumber = lastArbitratedTransactionNumberByArbitratorTable.get(status.getTransactionArbitrator());
-               if ((lastTransactionNumber != NULL) && (lastTransactionNumber >= status.getTransactionSequenceNumber())) {
+                       // Check if the transaction is dead
+                       if (lastArbitratedTransactionNumberByArbitratorTable->contains(status->getTransactionArbitrator()) && (lastArbitratedTransactionNumberByArbitratorTable->get(status->getTransactionArbitrator()) >= status->getTransactionSequenceNumber())) {
 
-                       // Set committed
-                       status.setStatus(TransactionStatus.StatusCommitted);
+                               // Set committed
+                               status->setStatus(TransactionStatus_StatusCommitted);
 
-                       // Remove
-                       iter.remove();
+                               // Remove
+                               iter->remove();
+                       }
                }
+               delete iter;
        }
 }
 
 /**
- * Process this slot, entry by entry.  Also update the latest message sent by slot
+ * Process this slot, entry by entry->  Also update the latest message sent by slot
  */
-void Table::processSlot(SlotIndexer indexer, Slot slot, bool acceptUpdatesToLocal, Hashset<int64_t> *machineSet) {
+void Table::processSlot(SlotIndexer *indexer, Slot *slot, bool acceptUpdatesToLocal, Hashset<int64_t> *machineSet) {
 
        // Update the last message seen
-       updateLastMessage(slot.getMachineID(), slot.getSequenceNumber(), slot, acceptUpdatesToLocal, machineSet);
+       updateLastMessage(slot->getMachineID(), slot->getSequenceNumber(), slot, acceptUpdatesToLocal, machineSet);
 
        // Process each entry in the slot
-       for (Entry entry : slot.getEntries()) {
-               switch (entry.getType()) {
-
-               case Entry.TypeCommitPart:
-                       processEntry((CommitPart)entry);
+       Vector<Entry *> *entries = slot->getEntries();
+       uint eSize = entries->size();
+       for (uint ei = 0; ei < eSize; ei++) {
+               Entry *entry = entries->get(ei);
+               switch (entry->getType()) {
+               case TypeCommitPart:
+                       processEntry((CommitPart *)entry);
                        break;
-
-               case Entry.TypeAbort:
-                       processEntry((Abort)entry);
+               case TypeAbort:
+                       processEntry((Abort *)entry);
                        break;
-
-               case Entry.TypeTransactionPart:
-                       processEntry((TransactionPart)entry);
+               case TypeTransactionPart:
+                       processEntry((TransactionPart *)entry);
                        break;
-
-               case Entry.TypeNewKey:
-                       processEntry((NewKey)entry);
+               case TypeNewKey:
+                       processEntry((NewKey *)entry);
                        break;
-
-               case Entry.TypeLastMessage:
-                       processEntry((LastMessage)entry, machineSet);
+               case TypeLastMessage:
+                       processEntry((LastMessage *)entry, machineSet);
                        break;
-
-               case Entry.TypeRejectedMessage:
-                       processEntry((RejectedMessage)entry, indexer);
+               case TypeRejectedMessage:
+                       processEntry((RejectedMessage *)entry, indexer);
                        break;
-
-               case Entry.TypeTableStatus:
-                       processEntry((TableStatus)entry, slot.getSequenceNumber());
+               case TypeTableStatus:
+                       processEntry((TableStatus *)entry, slot->getSequenceNumber());
                        break;
-
                default:
-                       throw new Error("Unrecognized type: " + entry.getType());
+                       throw new Error("Unrecognized type: ");
                }
        }
 }
@@ -2392,41 +2429,42 @@ void Table::processSlot(SlotIndexer indexer, Slot slot, bool acceptUpdatesToLoca
 /**
  * Update the last message that was sent for a machine Id
  */
-void Table::processEntry(LastMessage entry, Hashset<int64_t> *machineSet) {
+void Table::processEntry(LastMessage *entry, Hashset<int64_t> *machineSet) {
        // Update what the last message received by a machine was
-       updateLastMessage(entry.getMachineID(), entry.getSequenceNumber(), entry, false, machineSet);
+       updateLastMessage(entry->getMachineID(), entry->getSequenceNumber(), entry, false, machineSet);
 }
 
 /**
- * Add the new key to the arbitrators table and update the set of live new keys (in case of a rescued new key message)
+ * Add the new key to the arbitrators table and update the set of live
+ * new keys (in case of a rescued new key message)
  */
-void Table::processEntry(NewKey entry) {
-
+void Table::processEntry(NewKey *entry) {
        // Update the arbitrator table with the new key information
-       arbitratorTable.put(entry.getKey(), entry.getMachineID());
+       arbitratorTable->put(entry->getKey(), entry->getMachineID());
 
        // Update what the latest live new key is
-       NewKey oldNewKey = liveNewKeyTable.put(entry.getKey(), entry);
+       NewKey *oldNewKey = liveNewKeyTable->put(entry->getKey(), entry);
        if (oldNewKey != NULL) {
                // Delete the old new key messages
-               oldNewKey.setDead();
+               oldNewKey->setDead();
        }
 }
 
 /**
- * Process new table status entries and set dead the old ones as new ones come in.
- * keeps track of the largest and smallest table status seen in this current round
- * of updating the local copy of the block chain
+ * Process new table status entries and set dead the old ones as new
+ * ones come in-> keeps track of the largest and smallest table status
+ * seen in this current round of updating the local copy of the block
+ * chain
  */
-void Table::processEntry(TableStatus entry, int64_t seq) {
-       int newNumSlots = entry.getMaxSlots();
+void Table::processEntry(TableStatus *entry, int64_t seq) {
+       int newNumSlots = entry->getMaxSlots();
        updateCurrMaxSize(newNumSlots);
-
        initExpectedSize(seq, newNumSlots);
 
        if (liveTableStatus != NULL) {
-               // We have a larger table status so the old table status is no int64_ter alive
-               liveTableStatus.setDead();
+               // We have a larger table status so the old table status is no
+               // int64_ter alive
+               liveTableStatus->setDead();
        }
 
        // Make this new table status the latest alive table status
@@ -2434,297 +2472,285 @@ void Table::processEntry(TableStatus entry, int64_t seq) {
 }
 
 /**
- * Check old messages to see if there is a block chain violation. Also
+ * Check old messages to see if there is a block chain violation->
+ * Also
  */
-void Table::processEntry(RejectedMessage entry, SlotIndexer indexer) {
-       int64_t oldSeqNum = entry.getOldSeqNum();
-       int64_t newSeqNum = entry.getNewSeqNum();
-       bool isequal = entry.getEqual();
-       int64_t machineId = entry.getMachineID();
-       int64_t seq = entry.getSequenceNumber();
-
-
-       // Check if we have messages that were supposed to be rejected in our local block chain
+void Table::processEntry(RejectedMessage *entry, SlotIndexer *indexer) {
+       int64_t oldSeqNum = entry->getOldSeqNum();
+       int64_t newSeqNum = entry->getNewSeqNum();
+       bool isequal = entry->getEqual();
+       int64_t machineId = entry->getMachineID();
+       int64_t seq = entry->getSequenceNumber();
+
+       // Check if we have messages that were supposed to be rejected in
+       // our local block chain
        for (int64_t seqNum = oldSeqNum; seqNum <= newSeqNum; seqNum++) {
-
                // Get the slot
-               Slot slot = indexer.getSlot(seqNum);
+               Slot *slot = indexer->getSlot(seqNum);
 
                if (slot != NULL) {
-                       // If we have this slot make sure that it was not supposed to be a rejected slot
-
-                       int64_t slotMachineId = slot.getMachineID();
+                       // If we have this slot make sure that it was not supposed to be
+                       // a rejected slot
+                       int64_t slotMachineId = slot->getMachineID();
                        if (isequal != (slotMachineId == machineId)) {
-                               throw new Error("Server Error: Trying to insert rejected message for slot " + seqNum);
+                               throw new Error("Server Error: Trying to insert rejected message for slot ");
                        }
                }
        }
 
-
-       // Create a list of clients to watch until they see this rejected message entry.
+       // Create a list of clients to watch until they see this rejected
+       // message entry->
        Hashset<int64_t> *deviceWatchSet = new Hashset<int64_t>();
-       for (Map.Entry<int64_t Pair<int64_t Liveness> > lastMessageEntry : lastMessageTable.entrySet()) {
-
+       SetIterator<int64_t, Pair<int64_t, Liveness *> *> *iter = getKeyIterator(lastMessageTable);
+       while (iter->hasNext()) {
                // Machine ID for the last message entry
-               int64_t lastMessageEntryMachineId = lastMessageEntry.getKey();
+               int64_t lastMessageEntryMachineId = iter->next();
 
-               // We've seen it, don't need to continue to watch.  Our next
-               // message will implicitly acknowledge it.
+               // We've seen it, don't need to continue to watch->  Our next
+               // message will implicitly acknowledge it->
                if (lastMessageEntryMachineId == localMachineId) {
                        continue;
                }
 
-               Pair<int64_t Liveness> lastMessageValue = lastMessageEntry.getValue();
-               int64_t entrySequenceNumber = lastMessageValue.getFirst();
+               Pair<int64_t, Liveness *> *lastMessageValue = lastMessageTable->get(lastMessageEntryMachineId);
+               int64_t entrySequenceNumber = lastMessageValue->getFirst();
 
                if (entrySequenceNumber < seq) {
-
-                       // Add this rejected message to the set of messages that this machine ID did not see yet
+                       // Add this rejected message to the set of messages that this
+                       // machine ID did not see yet
                        addWatchVector(lastMessageEntryMachineId, entry);
-
-                       // This client did not see this rejected message yet so add it to the watch set to monitor
-                       deviceWatchSet.add(lastMessageEntryMachineId);
+                       // This client did not see this rejected message yet so add it
+                       // to the watch set to monitor
+                       deviceWatchSet->add(lastMessageEntryMachineId);
                }
        }
+       delete iter;
 
-       if (deviceWatchSet.isEmpty()) {
+       if (deviceWatchSet->isEmpty()) {
                // This rejected message has been seen by all the clients so
-               entry.setDead();
+               entry->setDead();
        } else {
                // We need to watch this rejected message
-               entry.setWatchSet(deviceWatchSet);
+               entry->setWatchSet(deviceWatchSet);
        }
 }
 
 /**
- * Check if this abort is live, if not then save it so we can kill it later.
- * update the last transaction number that was arbitrated on.
+ * Check if this abort is live, if not then save it so we can kill it
+ * later-> update the last transaction number that was arbitrated on->
  */
-void Table::processEntry(Abort entry) {
-
-
-       if (entry.getTransactionSequenceNumber() != -1) {
+void Table::processEntry(Abort *entry) {
+       if (entry->getTransactionSequenceNumber() != -1) {
                // update the transaction status if it was sent to the server
-               TransactionStatus status = outstandingTransactionStatus.remove(entry.getTransactionSequenceNumber());
+               TransactionStatus *status = outstandingTransactionStatus->remove(entry->getTransactionSequenceNumber());
                if (status != NULL) {
-                       status.setStatus(TransactionStatus.StatusAborted);
+                       status->setStatus(TransactionStatus_StatusAborted);
                }
        }
 
-       // Abort has not been seen by the client it is for yet so we need to keep track of it
-       Abort previouslySeenAbort = liveAbortTable.put(entry.getAbortId(), entry);
+       // Abort has not been seen by the client it is for yet so we need to
+       // keep track of it
+
+       Abort *previouslySeenAbort = liveAbortTable->put(new Pair<int64_t, int64_t>(entry->getAbortId()), entry);
        if (previouslySeenAbort != NULL) {
-               previouslySeenAbort.setDead();  // Delete old version of the abort since we got a rescued newer version
+               previouslySeenAbort->setDead();         // Delete old version of the abort since we got a rescued newer version
        }
 
-       if (entry.getTransactionArbitrator() == localMachineId) {
-               liveAbortsGeneratedByLocal.put(entry.getArbitratorLocalSequenceNumber(), entry);
+       if (entry->getTransactionArbitrator() == localMachineId) {
+               liveAbortsGeneratedByLocal->put(entry->getArbitratorLocalSequenceNumber(), entry);
        }
 
-       if ((entry.getSequenceNumber() != -1) && (lastMessageTable.get(entry.getTransactionMachineId()).getFirst() >= entry.getSequenceNumber())) {
-
+       if ((entry->getSequenceNumber() != -1) && (lastMessageTable->get(entry->getTransactionMachineId())->getFirst() >= entry->getSequenceNumber())) {
                // The machine already saw this so it is dead
-               entry.setDead();
-               liveAbortTable.remove(entry.getAbortId());
+               entry->setDead();
+               Pair<int64_t, int64_t> abortid = entry->getAbortId();
+               liveAbortTable->remove(&abortid);
 
-               if (entry.getTransactionArbitrator() == localMachineId) {
-                       liveAbortsGeneratedByLocal.remove(entry.getArbitratorLocalSequenceNumber());
+               if (entry->getTransactionArbitrator() == localMachineId) {
+                       liveAbortsGeneratedByLocal->remove(entry->getArbitratorLocalSequenceNumber());
                }
-
                return;
        }
 
-
-
-
        // Update the last arbitration data that we have seen so far
-       if (lastArbitrationDataLocalSequenceNumberSeenFromArbitrator.get(entry.getTransactionArbitrator()) != NULL) {
-
-               int64_t lastArbitrationSequenceNumber = lastArbitrationDataLocalSequenceNumberSeenFromArbitrator.get(entry.getTransactionArbitrator());
-               if (entry.getSequenceNumber() > lastArbitrationSequenceNumber) {
+       if (lastArbitrationDataLocalSequenceNumberSeenFromArbitrator->contains(entry->getTransactionArbitrator())) {
+               int64_t lastArbitrationSequenceNumber = lastArbitrationDataLocalSequenceNumberSeenFromArbitrator->get(entry->getTransactionArbitrator());
+               if (entry->getSequenceNumber() > lastArbitrationSequenceNumber) {
                        // Is larger
-                       lastArbitrationDataLocalSequenceNumberSeenFromArbitrator.put(entry.getTransactionArbitrator(), entry.getSequenceNumber());
+                       lastArbitrationDataLocalSequenceNumberSeenFromArbitrator->put(entry->getTransactionArbitrator(), entry->getSequenceNumber());
                }
        } else {
                // Never seen any data from this arbitrator so record the first one
-               lastArbitrationDataLocalSequenceNumberSeenFromArbitrator.put(entry.getTransactionArbitrator(), entry.getSequenceNumber());
+               lastArbitrationDataLocalSequenceNumberSeenFromArbitrator->put(entry->getTransactionArbitrator(), entry->getSequenceNumber());
        }
 
-
        // Set dead a transaction if we can
-       Transaction transactionToSetDead = liveTransactionByTransactionIdTable.remove(new Pair<int64_t, int64_t>(entry.getTransactionMachineId(), entry.getTransactionClientLocalSequenceNumber()));
+       Pair<int64_t, int64_t> deadPair = Pair<int64_t, int64_t>(entry->getTransactionMachineId(), entry->getTransactionClientLocalSequenceNumber());
+
+       Transaction *transactionToSetDead = liveTransactionByTransactionIdTable->remove(&deadPair);
        if (transactionToSetDead != NULL) {
-               liveTransactionBySequenceNumberTable.remove(transactionToSetDead.getSequenceNumber());
+               liveTransactionBySequenceNumberTable->remove(transactionToSetDead->getSequenceNumber());
        }
 
-       // Update the last transaction sequence number that the arbitrator arbitrated on
-       Long lastTransactionNumber = lastArbitratedTransactionNumberByArbitratorTable.get(entry.getTransactionArbitrator());
-       if ((lastTransactionNumber == NULL) || (lastTransactionNumber < entry.getTransactionSequenceNumber())) {
-
+       // Update the last transaction sequence number that the arbitrator
+       // arbitrated on
+       if (!lastArbitratedTransactionNumberByArbitratorTable->contains(entry->getTransactionArbitrator()) ||
+                       (lastArbitratedTransactionNumberByArbitratorTable->get(entry->getTransactionArbitrator()) < entry->getTransactionSequenceNumber())) {
                // Is a valid one
-               if (entry.getTransactionSequenceNumber() != -1) {
-                       lastArbitratedTransactionNumberByArbitratorTable.put(entry.getTransactionArbitrator(), entry.getTransactionSequenceNumber());
+               if (entry->getTransactionSequenceNumber() != -1) {
+                       lastArbitratedTransactionNumberByArbitratorTable->put(entry->getTransactionArbitrator(), entry->getTransactionSequenceNumber());
                }
        }
 }
 
 /**
- * Set dead the transaction part if that transaction is dead and keep track of all new parts
+ * Set dead the transaction part if that transaction is dead and keep
+ * track of all new parts
  */
-void Table::processEntry(TransactionPart entry) {
-       // Check if we have already seen this transaction and set it dead OR if it is not alive
-       Long lastTransactionNumber = lastArbitratedTransactionNumberByArbitratorTable.get(entry.getArbitratorId());
-       if ((lastTransactionNumber != NULL) && (lastTransactionNumber >= entry.getSequenceNumber())) {
+void Table::processEntry(TransactionPart *entry) {
+       // Check if we have already seen this transaction and set it dead OR
+       // if it is not alive
+       if (lastArbitratedTransactionNumberByArbitratorTable->contains(entry->getArbitratorId()) && (lastArbitratedTransactionNumberByArbitratorTable->get(entry->getArbitratorId()) >= entry->getSequenceNumber())) {
                // This transaction is dead, it was already committed or aborted
-               entry.setDead();
+               entry->setDead();
                return;
        }
 
        // This part is still alive
-       Hashtable<Pair<int64_t int32_t>, TransactionPart> transactionPart = newTransactionParts.get(entry.getMachineId());
+       Hashtable<Pair<int64_t, int32_t> *, TransactionPart *, uintptr_t, 0, pairHashFunction, pairEquals> *transactionPart = newTransactionParts->get(entry->getMachineId());
 
        if (transactionPart == NULL) {
                // Dont have a table for this machine Id yet so make one
-               transactionPart = new Hashtable<Pair<int64_t int32_t>, TransactionPart>();
-               newTransactionParts.put(entry.getMachineId(), transactionPart);
+               transactionPart = new Hashtable<Pair<int64_t, int32_t> *, TransactionPart *, uintptr_t, 0, pairHashFunction, pairEquals>();
+               newTransactionParts->put(entry->getMachineId(), transactionPart);
        }
 
-       // Update the part and set dead ones we have already seen (got a rescued version)
-       TransactionPart previouslySeenPart = transactionPart.put(entry.getPartId(), entry);
+       // Update the part and set dead ones we have already seen (got a
+       // rescued version)
+       TransactionPart *previouslySeenPart = transactionPart->put(new Pair<int64_t, int32_t>(entry->getPartId()), entry);
        if (previouslySeenPart != NULL) {
-               previouslySeenPart.setDead();
+               previouslySeenPart->setDead();
        }
 }
 
 /**
- * Process new commit entries and save them for future use.  Delete duplicates
+ * Process new commit entries and save them for future use->  Delete duplicates
  */
-void Table::processEntry(CommitPart entry) {
-
-
+void Table::processEntry(CommitPart *entry) {
        // Update the last transaction that was updated if we can
-       if (entry.getTransactionSequenceNumber() != -1) {
-               Long lastTransactionNumber = lastArbitratedTransactionNumberByArbitratorTable.get(entry.getMachineId());
-
-               // Update the last transaction sequence number that the arbitrator arbitrated on
-               if ((lastTransactionNumber == NULL) || (lastTransactionNumber < entry.getTransactionSequenceNumber())) {
-                       lastArbitratedTransactionNumberByArbitratorTable.put(entry.getMachineId(), entry.getTransactionSequenceNumber());
+       if (entry->getTransactionSequenceNumber() != -1) {
+               if (!lastArbitratedTransactionNumberByArbitratorTable->contains(entry->getMachineId() || lastArbitratedTransactionNumberByArbitratorTable->get(entry->getMachineId()) < entry->getTransactionSequenceNumber())) {
+                       lastArbitratedTransactionNumberByArbitratorTable->put(entry->getMachineId(), entry->getTransactionSequenceNumber());
                }
        }
 
-
-
-
-       Hashtable<Pair<int64_t int32_t>, CommitPart> commitPart = newCommitParts.get(entry.getMachineId());
-
+       Hashtable<Pair<int64_t, int32_t> *, CommitPart *, uintptr_t, 0, pairHashFunction, pairEquals> *commitPart = newCommitParts->get(entry->getMachineId());
        if (commitPart == NULL) {
                // Don't have a table for this machine Id yet so make one
-               commitPart = new Hashtable<Pair<int64_t int32_t>, CommitPart>();
-               newCommitParts.put(entry.getMachineId(), commitPart);
+               commitPart = new Hashtable<Pair<int64_t, int32_t> *, CommitPart *, uintptr_t, 0, pairHashFunction, pairEquals>();
+               newCommitParts->put(entry->getMachineId(), commitPart);
        }
-
-       // Update the part and set dead ones we have already seen (got a rescued version)
-       CommitPart previouslySeenPart = commitPart.put(entry.getPartId(), entry);
+       // Update the part and set dead ones we have already seen (got a
+       // rescued version)
+       CommitPart *previouslySeenPart = commitPart->put(new Pair<int64_t, int32_t>(entry->getPartId()), entry);
        if (previouslySeenPart != NULL) {
-               previouslySeenPart.setDead();
+               previouslySeenPart->setDead();
        }
 }
 
 /**
- * Update the last message seen table.  Update and set dead the appropriate RejectedMessages as clients see them.
- * Updates the live aborts, removes those that are dead and sets them dead.
- * Check that the last message seen is correct and that there is no mismatch of our own last message or that
- * other clients have not had a rollback on the last message.
+ * Update the last message seen table-> Update and set dead the
+ * appropriate RejectedMessages as clients see them-> Updates the live
+ * aborts, removes those that are dead and sets them dead-> Check that
+ * the last message seen is correct and that there is no mismatch of
+ * our own last message or that other clients have not had a rollback
+ * on the last message->
  */
-void Table::updateLastMessage(int64_t machineId, int64_t seqNum, Liveness liveness, bool acceptUpdatesToLocal, Hashset<intr64_t> *machineSet) {
-
+void Table::updateLastMessage(int64_t machineId, int64_t seqNum, Liveness *liveness, bool acceptUpdatesToLocal, Hashset<int64_t> *machineSet) {
        // We have seen this machine ID
-       machineSet.remove(machineId);
+       machineSet->remove(machineId);
 
        // Get the set of rejected messages that this machine Id is has not seen yet
-       Hashset<RejectedMessage *> *watchset = rejectedMessageWatchVectorTable.get(machineId);
-
+       Hashset<RejectedMessage *> *watchset = rejectedMessageWatchVectorTable->get(machineId);
        // If there is a rejected message that this machine Id has not seen yet
        if (watchset != NULL) {
-
-               // Go through each rejected message that this machine Id has not seen yet
-               for (Iterator<RejectedMessage> rmit = watchset.iterator(); rmit.hasNext(); ) {
-
-                       RejectedMessage rm = rmit.next();
-
-                       // If this machine Id has seen this rejected message...
-                       if (rm.getSequenceNumber() <= seqNum) {
-
+               // Go through each rejected message that this machine Id has not
+               // seen yet
+
+               SetIterator<RejectedMessage *, RejectedMessage *> *rmit = watchset->iterator();
+               while (rmit->hasNext()) {
+                       RejectedMessage *rm = rmit->next();
+                       // If this machine Id has seen this rejected message->->->
+                       if (rm->getSequenceNumber() <= seqNum) {
                                // Remove it from our watchlist
-                               rmit.remove();
-
+                               rmit->remove();
                                // Decrement machines that need to see this notification
-                               rm.removeWatcher(machineId);
+                               rm->removeWatcher(machineId);
                        }
                }
+               delete rmit;
        }
 
        // Set dead the abort
-       for (Iterator<Map.Entry<Pair<int64_t, int64_t>, Abort> > i = liveAbortTable.entrySet().iterator(); i.hasNext();) {
-               Abort abort = i.next().getValue();
-
-               if ((abort.getTransactionMachineId() == machineId) && (abort.getSequenceNumber() <= seqNum)) {
-                       abort.setDead();
-                       i.remove();
-
-                       if (abort.getTransactionArbitrator() == localMachineId) {
-                               liveAbortsGeneratedByLocal.remove(abort.getArbitratorLocalSequenceNumber());
+       SetIterator<Pair<int64_t, int64_t> *, Abort *, uintptr_t, 0, pairHashFunction, pairEquals> *abortit = getKeyIterator(liveAbortTable);
+
+       while (abortit->hasNext()) {
+               Pair<int64_t, int64_t> *key = abortit->next();
+               Abort *abort = liveAbortTable->get(key);
+               if ((abort->getTransactionMachineId() == machineId) && (abort->getSequenceNumber() <= seqNum)) {
+                       abort->setDead();
+                       abortit->remove();
+                       if (abort->getTransactionArbitrator() == localMachineId) {
+                               liveAbortsGeneratedByLocal->remove(abort->getArbitratorLocalSequenceNumber());
                        }
                }
        }
-
-
-
+       delete abortit;
        if (machineId == localMachineId) {
-               // Our own messages are immediately dead.
-               if (liveness instanceof LastMessage) {
-                       ((LastMessage)liveness).setDead();
-               } else if (liveness instanceof Slot) {
-                       ((Slot)liveness).setDead();
+               // Our own messages are immediately dead->
+               char livenessType = liveness->getType();
+               if (livenessType == TypeLastMessage) {
+                       ((LastMessage *)liveness)->setDead();
+               } else if (livenessType == TypeSlot) {
+                       ((Slot *)liveness)->setDead();
                } else {
                        throw new Error("Unrecognized type");
                }
        }
-
        // Get the old last message for this device
-       Pair<int64_t Liveness> lastMessageEntry = lastMessageTable.put(machineId, new Pair<int64_t Liveness>(seqNum, liveness));
+       Pair<int64_t, Liveness *> *lastMessageEntry = lastMessageTable->put(machineId, new Pair<int64_t, Liveness *>(seqNum, liveness));
        if (lastMessageEntry == NULL) {
                // If no last message then there is nothing else to process
                return;
        }
 
-       int64_t lastMessageSeqNum = lastMessageEntry.getFirst();
-       Liveness lastEntry = lastMessageEntry.getSecond();
+       int64_t lastMessageSeqNum = lastMessageEntry->getFirst();
+       Liveness *lastEntry = lastMessageEntry->getSecond();
+       delete lastMessageEntry;
 
        // If it is not our machine Id since we already set ours to dead
        if (machineId != localMachineId) {
-               if (lastEntry instanceof LastMessage) {
-                       ((LastMessage)lastEntry).setDead();
-               } else if (lastEntry instanceof Slot) {
-                       ((Slot)lastEntry).setDead();
+               char lastEntryType = lastEntry->getType();
+
+               if (lastEntryType == TypeLastMessage) {
+                       ((LastMessage *)lastEntry)->setDead();
+               } else if (lastEntryType == TypeSlot) {
+                       ((Slot *)lastEntry)->setDead();
                } else {
                        throw new Error("Unrecognized type");
                }
        }
-
        // Make sure the server is not playing any games
        if (machineId == localMachineId) {
-
                if (hadPartialSendToServer) {
                        // We were not making any updates and we had a machine mismatch
                        if (lastMessageSeqNum > seqNum && !acceptUpdatesToLocal) {
-                               throw new Error("Server Error: Mismatch on local machine sequence number, needed at least: " +  lastMessageSeqNum  + " got: " + seqNum);
+                               throw new Error("Server Error: Mismatch on local machine sequence number, needed at least: ");
                        }
-
                } else {
                        // We were not making any updates and we had a machine mismatch
                        if (lastMessageSeqNum != seqNum && !acceptUpdatesToLocal) {
-                               throw new Error("Server Error: Mismatch on local machine sequence number, needed: " +  lastMessageSeqNum + " got: " + seqNum);
+                               throw new Error("Server Error: Mismatch on local machine sequence number, needed: ");
                        }
                }
        } else {
@@ -2735,29 +2761,29 @@ void Table::updateLastMessage(int64_t machineId, int64_t seqNum, Liveness livene
 }
 
 /**
- * Add a rejected message entry to the watch set to keep track of which clients have seen that
- * rejected message entry and which have not.
+ * Add a rejected message entry to the watch set to keep track of
+ * which clients have seen that rejected message entry and which have
+ * not.
  */
-void Table::addWatchVector(int64_t machineId, RejectedMessage entry) {
-       Hashset<RejectedMessage *> *entries = rejectedMessageWatchVectorTable.get(machineId);
+void Table::addWatchVector(int64_t machineId, RejectedMessage *entry) {
+       Hashset<RejectedMessage *> *entries = rejectedMessageWatchVectorTable->get(machineId);
        if (entries == NULL) {
                // There is no set for this machine ID yet so create one
                entries = new Hashset<RejectedMessage *>();
-               rejectedMessageWatchVectorTable.put(machineId, entries);
+               rejectedMessageWatchVectorTable->put(machineId, entries);
        }
-       entries.add(entry);
+       entries->add(entry);
 }
 
 /**
  * Check if the HMAC chain is not violated
  */
-void Table::checkHMACChain(SlotIndexer indexer, Array<Slot> *newSlots) {
-       for (int i = 0; i < newSlots.length; i++) {
-               Slot currSlot = newSlots[i];
-               Slot prevSlot = indexer.getSlot(currSlot.getSequenceNumber() - 1);
+void Table::checkHMACChain(SlotIndexer *indexer, Array<Slot *> *newSlots) {
+       for (int i = 0; i < newSlots->length(); i++) {
+               Slot *currSlot = newSlots->get(i);
+               Slot *prevSlot = indexer->getSlot(currSlot->getSequenceNumber() - 1);
                if (prevSlot != NULL &&
-                               !Arrays.equals(prevSlot.getHMAC(), currSlot.getPrevHMAC()))
-                       throw new Error("Server Error: Invalid HMAC Chain" + currSlot + " " + prevSlot);
+                               !prevSlot->getHMAC()->equals(currSlot->getPrevHMAC()))
+                       throw new Error("Server Error: Invalid HMAC Chain");
        }
 }
-