Fruzhin icon indicating copy to clipboard operation
Fruzhin copied to clipboard

Transaction queue

Open ablax opened this issue 2 years ago • 2 comments

  • [x] Create transaction pool
  • [x] Create transaction queue
  • [x] Prepare for transaction validation

ablax avatar Nov 22 '23 13:11 ablax

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
0.0% 0.0% Duplication

sonarqubecloud[bot] avatar Nov 29 '23 08:11 sonarqubecloud[bot]

Currently blocked, as it depends on Trie

ablax avatar Jan 19 '24 08:01 ablax

This PR can be merged with its current state. The logic for sending a validation request to the runtime and decoding the response is there. However, it seems we have some kind of an issue that causes the runtime to respond with an invalid transaction validation error for every transaction.

We have:

  • Logic for validating incoming transaction from the transactions protocol.
  • Logic for appropriately putting transactions in queue or pool.

What is still needed:

  • Fix the issue that causes validation error. (could be related to the fact that we don't execute blocks after full sync, or as Kamil from Kagome mentioned it could be because the validation runtime request is trying to alter the state)
  • Send known non repetitive transactions to peers in a transaction message.

Zurcusa avatar Oct 21 '24 14:10 Zurcusa