Some minor LND updates mostly etcd related#6
Open
bhandras wants to merge 3 commits intobottlepay:masterfrom
Open
Some minor LND updates mostly etcd related#6bhandras wants to merge 3 commits intobottlepay:masterfrom
bhandras wants to merge 3 commits intobottlepay:masterfrom
Conversation
Contributor
|
Hi Andras, definitely a fair point about TLS. I ran your branch on our benchmarking rig (hardware described in the repo readme) and got the following result: The previous result of 4 tps was obtained after 10k payments. I see that it went up to 6 tps. One thing that I was wondering is whether there are also other changes in lnd 0.13 that increase etcd performance? When the final release comes out, we can merge this PR and also update the remaining lnd configurations to 0.13. Regarding etcd, it still seems to be the case that performance goes way down with larger number of payments, whereas with postgres tps remains stable. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi Bottlepay,
Ran a few benchmarks and for the etcd case it seems like a bunch CPU of time is spent in the TLS layer. I thought it'd be an interesting comparison vs the experimental postgres kvdb driver since it also doesn't use TLS at the moment. PTAL.
For reference, on my Hetzner cloud instance (AX-41-NVMe) the first few data points are: