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

Memory usage tune-up #908

Merged
merged 2 commits into from
Jun 18, 2019
Merged

Memory usage tune-up #908

merged 2 commits into from
Jun 18, 2019

Conversation

AionJayT
Copy link
Collaborator

Notice

It is not allowed to submit your PR to the 'master' branch directly, please submit your PR to the 'master-pre-merge' branch and rebase your PR to 'master-pre-merge' branch.

Description

Please include a brief summary of the change that this pull request proposes. Include any relevant motivation and context. List any dependencies required for this change.

  • Kernel memory usage tune-up. The aionblock cached the unnecessary trie instance after calculated the transaction state root. Therefore, when the syncMgr queuing the downloaded blocks with tons of transactions from the network. It will consume a lot of memory. It should be released right away.
  • Changed the Linux kernel env setup relate with glibc for solving the memory occupy issue.

Fixes Issue # .

Type of change

Insert x into the following checkboxes to confirm (eg. [x]):

  • Bug fix.
  • New feature.
  • Enhancement.
  • Unit test.
  • Breaking change (a fix or feature that causes existing functionality to not work as expected).
  • Requires documentation update.

Testing

Please describe the tests you used to validate this pull request. Provide any relevant details for test configurations as well as any instructions to reproduce these results.

  • syncing test with the network. The kernel stays around 4GB after tune-up even import the heavy transaction blocks (before is 7~8GB).

@AionJayT AionJayT added the enhancement New feature or request label Jun 14, 2019
@AionJayT AionJayT added this to the 0.4.1 milestone Jun 14, 2019
@AionJayT AionJayT requested a review from AlexandraRoatis June 14, 2019 01:33
@AionJayT AionJayT changed the title AkI-145 Memory usuage tune-up Jun 14, 2019
@AionJayT AionJayT changed the title Memory usuage tune-up Memory usage tune-up Jun 14, 2019
AionJayT added 2 commits June 17, 2019 17:57
…Mgr by removing the trie instance in the AionBlock 2. tune up the txValidator chached tx size
@AionJayT AionJayT merged commit d0a02ba into master-pre-merge Jun 18, 2019
@AionJayT AionJayT deleted the AKI-145 branch June 19, 2019 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants