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

loopd: allow setting RPC batch size when running cost migration #829

Merged

Conversation

bhandras
Copy link
Member

Previously we defaulted to fetching 1000 payments at a time when running the cost migration however due to slowness on LND side this may prevent the migration from completing due to RPC timeout kicking in. This commit adds a new config value to give more control to the user by setting the batch size as needed.

Pull Request Checklist

  • Update release_notes.md if your PR contains major features, breaking changes or bugfixes

Previously we defaulted to fetching 1000 payments at a time when running
the cost migration however due to slowness on LND side this may prevent
the migration from completing due to RPC timeout kicking in. This commit
adds a new config value to give more control to the user by setting the
batch size as needed.
Copy link
Member

@sputn1ck sputn1ck left a comment

Choose a reason for hiding this comment

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

LGTM! thanks for the fix

@djkazic
Copy link
Collaborator

djkazic commented Sep 20, 2024

LGTM, got my loopd back online with this and batch size = 10 (postgres_kvdb backend on LND)

@hieblmi hieblmi self-requested a review October 8, 2024 12:52
Copy link
Collaborator

@hieblmi hieblmi left a comment

Choose a reason for hiding this comment

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

LGTM!

@bhandras bhandras merged commit 61c0b45 into lightninglabs:master Oct 8, 2024
4 checks passed
@bhandras bhandras deleted the cost-migration-rpc-batchsize branch October 8, 2024 12:55
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.

4 participants