-
Notifications
You must be signed in to change notification settings - Fork 70
Define listeners for transaction pool related events - Closes #873 #917
Define listeners for transaction pool related events - Closes #873 #917
Conversation
…n interface for lisk-transaction-pool
…optional for transaction interface for lisk-transaction-pool
7c6de95
to
97a619d
Compare
|
||
private existsInTransactionPool(transaction: Transaction): boolean {} | ||
public getProcessableTransactions(limit: number): ReadonlyArray<Transaction> { | ||
return this._queues.ready.dequeueUntil( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wasnt the idea to keep it in the queue until we removeConfirmedTransaction
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This function is not just the basic implementation, I'll update this function I implement this feature.
verifyTransactionAgainstOtherTransactions?( | ||
transaction: Transaction, | ||
containsUniqueData(): boolean; | ||
verifyTransactionAgainstOtherTransactions( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Leaving a comment here for future naming revision: verifyTransactionAgainstOtherTransactions
-> verifyAgainstOtherTransactions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
small typo
.map(queueName => this._queues[queueName].removeFor(condition)) | ||
.reduce( | ||
( | ||
transactionsAccumelatedFromQueues: ReadonlyArray<Transaction>, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
transactionsAccumelatedFromQueues
-> transactionsAccumulatedFromQueues
Add blockchain state change event listeners for the transaction pool.