Tech: update protobuf and db schema by adding mempool, transaction, account structure #20
Comments
This was referenced Jul 4, 2019
iltoga
pushed a commit
that referenced
this issue
Jul 9, 2019
andy-shi88
pushed a commit
that referenced
this issue
Jul 10, 2019
astaphobia
pushed a commit
that referenced
this issue
Jul 17, 2019
* #20 added timestamp to transaction table and simplified mempool table structure * #14 - WIP: started implementing mempoolService * #14 - WIP: still implementing mempoolservice and mempoolQuery * fixed GetMempoolTransaction query * #6 add executeTransactionStatements method to handle multiple query statement * #6 add get transaction bytes and parse transaction bytes method * #6 update transaction model * #6 handle no-recipient get transaction bytes * #6 update doc : add transaction bytes structure * implementation of mempoolCoreService * #14 tested GetMempoolTransactions, AddMempoolTransaction, RemoveMempoolTransaction * #14 completed testing mempool service * start integrating mempool service into main process and blockchainprocessor * added tx timeout (mempool tx expiration time) * added getamount and getsize to transaction,TypeAction added logic to pass in payload to blockchainprocessor when generating blocks * small variable name refactoring, as suggested by code linter, to avoid confusion * recompiled schema * fixed comments * removed unused interface signature * added missing interface signature * fixed logic error in tx GetSize for SendMoney tx (we are getting only body length, not full tx length) * fixed some lint issues * refactored part of mempoolService: moved removemempooltransactions to blockservice because part of PushBlock method fixed unit tests added to transactionUtil function to calculate transaction ID from transaction model * refactored old unit tests (64 bytes signature instead of 256) added tests for GetTransactionID * merged develop with branch fixed parse transaction bytes: added checks if bytes we are parsing are formally correct while reading them * merged develop into branch * resolved some of the comments from the PR fixed relative tests * added transactionQuery interface to blockservice and added insertTransaction to pushblock * fixed NewBlock: was missing payloadlength fixed bug in GenerateBlock: wrong variable scope fixed unit tests * removed comment relative to mempool table name
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
Add mempool, transaction, account tables and relative protobuf interfaces in respect of the design issue #1, #10, #13
Breakdown
Additional Diagram / File
Dependencies
this partially depends on: #7 and #18
The text was updated successfully, but these errors were encountered: