Skip to content
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.

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

Enhance AH rocksdb-prototype to cover all exiting functionality of account-history plugin #2066

Closed
vogel76 opened this issue Jan 31, 2018 · 2 comments
Assignees

Comments

@vogel76
Copy link
Contributor

vogel76 commented Jan 31, 2018

Add support for:

  • account white-listing, black-listing
  • operation filtering
  • data pruning to cover all exiting functionality of account-history plugin
@vogel76 vogel76 self-assigned this Jan 31, 2018
vogel76 added a commit that referenced this issue Feb 1, 2018
- Implemented same options as in old AH plugin. Options in old AH plugin renamed to avoid conflict.
- Implemented filtering data collection (for account filtering).
Not finished yet. Can fail CI.
vogel76 added a commit that referenced this issue Feb 4, 2018
…hanged data structures to allow counting AH records for given account (similary to original implementation).
vogel76 added a commit that referenced this issue Feb 7, 2018
…n of AH data prunning. Reimplemented get_account_history API method accordingly.
@vogel76
Copy link
Contributor Author

vogel76 commented Feb 9, 2018

Added previously missing functionality:

  • account filtering
  • operation filtering
  • AH data pruning
    Needed changes did force redesign of data structures and it lead to small slowdown according to golden results initially taken (up to 19M blocks):
  • reindex process is a little longer (reindex slowdown is near to 10%) and takes now 6678s (6065s originally, and 13328 in develop) what is caused by directly reading data while writing them into db.
  • reindex process is significantly longer when prune is enabled and takes 10411s. Comparison will be added when reindex + prune will complete on develop version (it is longer than no-prune).
  • memory allocation is stable, depends on cache configuration.
    To make PR, docker build configuration must be fixed, what will be next step to be done.

@vogel76
Copy link
Contributor Author

vogel76 commented Feb 9, 2018

As a side note, prune option by default shall be disabled now. It damages historical data, makes data collection longer. So it makes no sense to have it enabled by default

vogel76 added a commit that referenced this issue Mar 7, 2018
vogel76 added a commit that referenced this issue Mar 7, 2018
mvandeberg pushed a commit that referenced this issue Mar 20, 2018
- Implemented same options as in old AH plugin. Options in old AH plugin renamed to avoid conflict.
- Implemented filtering data collection (for account filtering).
Not finished yet. Can fail CI.
mvandeberg pushed a commit that referenced this issue Mar 20, 2018
…hanged data structures to allow counting AH records for given account (similary to original implementation).
mvandeberg pushed a commit that referenced this issue Mar 20, 2018
…n of AH data prunning. Reimplemented get_account_history API method accordingly.
mvandeberg pushed a commit that referenced this issue Mar 20, 2018
mvandeberg pushed a commit that referenced this issue Mar 20, 2018
mvandeberg added a commit that referenced this issue Mar 20, 2018
mvandeberg pushed a commit that referenced this issue Mar 21, 2018
- Implemented same options as in old AH plugin. Options in old AH plugin renamed to avoid conflict.
- Implemented filtering data collection (for account filtering).
Not finished yet. Can fail CI.
mvandeberg pushed a commit that referenced this issue Mar 21, 2018
…hanged data structures to allow counting AH records for given account (similary to original implementation).
mvandeberg pushed a commit that referenced this issue Mar 21, 2018
…n of AH data prunning. Reimplemented get_account_history API method accordingly.
mvandeberg pushed a commit that referenced this issue Mar 21, 2018
mvandeberg pushed a commit that referenced this issue Mar 21, 2018
mvandeberg added a commit that referenced this issue Mar 21, 2018
mvandeberg added a commit that referenced this issue Mar 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants