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

channeldb: make channel graph population async #6187

Open
Roasbeef opened this issue Jan 21, 2022 · 1 comment
Open

channeldb: make channel graph population async #6187

Roasbeef opened this issue Jan 21, 2022 · 1 comment
Labels
graph intermediate Issues suitable for developers moderately familiar with the codebase and LN optimization P2 should be fixed if one has time

Comments

@Roasbeef
Copy link
Member

In #6111 we significantly sped up the population of the channel graph cache. However, as is, it's still fully blocking: lnd won't start up until the cache is populated. However there's no reason for this to be the case, as things like forwarding HTLCs don't actually require access to the cache. Instead, we should modify things such that lnd can start up as soon as the graph is open, and callers block until cache population is complete.
@Roasbeef Roasbeef added intermediate Issues suitable for developers moderately familiar with the codebase and LN graph optimization labels Jan 21, 2022
@Roasbeef
Copy link
Member Author

Related to #6107 (comment)

@HannahMR HannahMR added the P2 should be fixed if one has time label Jan 21, 2022
@Roasbeef Roasbeef added this to the v0.16.0 milestone Apr 18, 2022
@Roasbeef Roasbeef mentioned this issue Jun 21, 2022
Open
@Roasbeef Roasbeef modified the milestones: v0.16.0, v0.17.0 Aug 23, 2022
@saubyk saubyk removed this from the Low Priority milestone Aug 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
graph intermediate Issues suitable for developers moderately familiar with the codebase and LN optimization P2 should be fixed if one has time
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Roasbeef @HannahMR @saubyk