Skip to content

add list of transactions concerning account to chainstate (block_hash, tx_hash) #91

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
2 tasks done
thepiwo opened this issue Oct 25, 2017 · 0 comments · Fixed by #107
Closed
2 tasks done

add list of transactions concerning account to chainstate (block_hash, tx_hash) #91

thepiwo opened this issue Oct 25, 2017 · 0 comments · Fixed by #107
Assignees
@d-velev

Comments

@thepiwo
Copy link
Collaborator

thepiwo commented Oct 25, 2017 •
edited by d-velev
Loading

  • Keep the chain_state as it is. %{acc => %{balance: balance, latest_nonce: latest nonce}}
  • make a txs_index which is %{acc => [{block_hash, tx_hash}]} this gets updated on add_block but is not kept in the block itself
@d-velev d-velev self-assigned this Oct 31, 2017
@thepiwo thepiwo mentioned this issue Nov 1, 2017
Closed
2 tasks
d-velev added a commit that referenced this issue Nov 1, 2017
@d-velev
[GH-#91] Helper functions added to chain module, refactoring not yet …
ad06f2d 
…done
d-velev added a commit that referenced this issue Nov 2, 2017
@d-velev
[GH-#91] State of the Chain GenServer now also contains a map with ac…
3bbe48e 
…count as key and a list of {block_hash, tx_hash} for each tx as value
@d-velev d-velev mentioned this issue Nov 2, 2017
Merged
d-velev added a commit that referenced this issue Nov 2, 2017
@d-velev
[GH-#91] Fixed issues after merge with master
f647551
d-velev added a commit that referenced this issue Nov 2, 2017
@d-velev
[GH-#91] Initial txs_index is calculated with the genesis block (stil…
55579c6 
…l an empty map)
d-velev added a commit that referenced this issue Nov 2, 2017
@d-velev
[GH-#91] Function in Chain renamed
4b66b5c
thepiwo added a commit that referenced this issue Nov 2, 2017
@thepiwo
Merge pull request #107 from aeternity/GH-91
f553cd5 
GH-91 Add list of transactions concerning account to chainstate (block_hash, tx_hash)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@d-velev d-velev

Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@d-velev @thepiwo