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

Performance Regression in Geth v1.10.21 #25432

Closed
gabronick opened this issue Jul 28, 2022 · 3 comments
Closed

Performance Regression in Geth v1.10.21 #25432

gabronick opened this issue Jul 28, 2022 · 3 comments
Labels

Comments

@gabronick
Copy link

gabronick commented Jul 28, 2022

System information

Geth version: Geth v1.10.21
OS & Version: Linux Amazon EC2 5.10.130-118.517.amzn2.aarch64

Expected behaviour

blocks are processed very fast, several blocks usually take a few seconds, the server runs on NVME

example:

Imported new chain segment blocks=76 txs=12404 mgas=1108.491 elapsed=8.108s mgasps=136.710 number=15,197,808 hash=aab71b..b5f1e8 age=5d11h4m dirty=123.00MiB

Actual behaviour

Imported new chain segment blocks=5 txs=1250 mgas=115.102 elapsed=59.160s mgasps=1.946 number=15,232,749 hash=3fd81a..e5dd9d age=16m36s dirty=5.14GiB

the blocks start to process so slowly that the node lags, the only way is to restart the node

Steps to reproduce the behaviour

let the node run for a while,

UPDATE:

after going back to version .20 the stability has improved, we have never seen this behavior before and the workload is constant

@ligi
Copy link
Member

ligi commented Aug 4, 2022

Thanks for the report!

Can you let us now the command line parameters that you start geth with?

It may be related to this change:

We have reverted to an older version of goleveldb because recent versions have buggy compaction and manifest handling. (#25413)

but needs more investigation.

@fjl
Copy link
Contributor

fjl commented Aug 4, 2022

A theory why it could be slow:

In geth v1.10.20, we upgraded to a broken version of leveldb which may skip database compaction in some cases. When you upgrade to .21, it could be that leveldb needs to perform significant compaction work, which would reduce performance until that's done.

@fjl fjl removed the status:triage label Aug 4, 2022
@fjl fjl changed the title Performance Regresion Performance Regression in Geth v1.10.21 Aug 4, 2022
@MariusVanDerWijden
Copy link
Member

Closing this for now, since it went pretty stale. If you still see these performance regressions with v1.10.25 please open another issue. thanks for reporting

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants
@fjl @ligi @MariusVanDerWijden @gabronick and others