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

Fix account_tx sluggishness #308

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

Fix account_tx sluggishness #308

wants to merge 2 commits into from

Conversation

WietseWind
Copy link
Contributor

Implements fix as per XRPLF/rippled@2e9261cb

  • Pre fix: ignoring index, doing full table scan. Problematic on accounts with many records / large DB (FH server).

As per Ripple Eng:

The SQLite db is not using indexes with the previous SQL, so full table scan was going on. The fh nodes has a big table so the thread is hanging
I found the issue but it’s the developer who found the SQLite optimizer problem
It only happens when the account has a lot of transactions and marker is used for paging

@WietseWind
Copy link
Contributor Author

WietseWind commented May 30, 2024

Already merged into jshooks; - will reach DEV branch that way too.
#330

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

Successfully merging this pull request may close these issues.

2 participants