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.

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

chore: change default cache size config #1531

Merged
merged 1 commit into from
Nov 7, 2023
Merged

chore: change default cache size config #1531

merged 1 commit into from
Nov 7, 2023

Conversation

driftluo
Copy link
Contributor

@driftluo driftluo commented Nov 7, 2023 •
edited by Flouse
Loading

What this PR does / why we need it?

rocksdb_cache trie_cache Memory Usage TPS
0 0    
10 20 1.31G 745
30 50 1.43G 747
50 100 1.37G 825
100 200 1.52G 1000
200 500 1.54 1004

What is the impact of this PR?

No Breaking Change

PR relation:

CI Settings

CI Usage

Tip: Check the CI you want to run below, and then comment /run-ci.

CI Switch

  • Web3 Compatible Tests
  • OCT 1-5 And 12-15
  • OCT 6-10
  • OCT 11
  • OCT 16-19
  • v3 Core Tests

CI Description

CI Name Description
Web3 Compatible Test Test the Web3 compatibility of Axon
v3 Core Test Run the compatibility tests provided by Uniswap V3
OCT 1-5 | 6-10 | 11 | 12-15 | 16-19 Run the compatibility tests provided by OpenZeppelin

@driftluo driftluo requested a review from a team as a code owner November 7, 2023 05:48
@driftluo driftluo requested review from ashuralyk and blckngm November 7, 2023 05:48
@github-actions github-actions bot added the chore label Nov 7, 2023
yangby-cryptape
yangby-cryptape previously approved these changes Nov 7, 2023
View reviewed changes
@KaoImin

This comment was marked as resolved.

Sign in to view
@driftluo driftluo force-pushed the change-default-config branch from 57e04c5 to eb0271d Compare November 7, 2023 07:17
@KaoImin KaoImin changed the title chore: change default config chore: change default cache size config Nov 7, 2023
@yangby-cryptape yangby-cryptape added this pull request to the merge queue Nov 7, 2023
Merged via the queue into main with commit 64f07fb Nov 7, 2023
23 checks passed
@Flouse Flouse linked an issue Nov 7, 2023 that may be closed by this pull request
Axon node memory consumption #1289
Closed
Flouse
Flouse reviewed Nov 7, 2023
View reviewed changes
Copy link
Contributor

@Flouse Flouse left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please update the document too.
https://docs.axonweb3.io/devops/configuration/#triedb_cache_size

This was referenced Nov 8, 2023
Merged
Closed
Closed
KaoImin pushed a commit that referenced this pull request Nov 17, 2023
@driftluo @KaoImin
chore: change default cache size config (#1531)
9a815e0 
chore: change default config
@driftluo driftluo deleted the change-default-config branch November 21, 2023 06:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Flouse Flouse Flouse left review comments

@yangby-cryptape yangby-cryptape yangby-cryptape approved these changes

@KaoImin KaoImin KaoImin approved these changes

@ashuralyk ashuralyk Awaiting requested review from ashuralyk ashuralyk is a code owner automatically assigned from axonweb3/axon-dev

@blckngm blckngm Awaiting requested review from blckngm blckngm is a code owner automatically assigned from axonweb3/axon-dev

Assignees
No one assigned
Labels
chore
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Axon node memory consumption
4 participants
@driftluo @KaoImin @Flouse @yangby-cryptape