Memory Leak: TransactionTable never cleans up complete transactions

When a transaction commits, we mark it as completed in markCompletedTransaction(). The issue is this map is never cleaned up – there is no call to cleanupCompletedTransactions().

(Was this supposed to replace the StaleTransactionCleanupService?)

So the transaction (and address and UUID object) leak for every transaction, until the node OOMs.