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

Tracking issue: kvdb/postgres-Remove global application level lock #9229

Open
saubyk opened this issue Oct 28, 2024 · 0 comments · May be fixed by #9242
Open

Tracking issue: kvdb/postgres-Remove global application level lock #9229

saubyk opened this issue Oct 28, 2024 · 0 comments · May be fixed by #9242
Assignees
@bhandras
Labels
database Related to the database/storage of LND enhancement Improvements to existing features / behaviour kvdb postgres
Milestone
v0.19.0

Comments

@saubyk
Copy link
Collaborator

saubyk commented Oct 28, 2024

Need to remove the global application level lock from the postgres backend. This lock prevents multiple write transactions from happening at the same time, and will also block a writer if a read is on going.

There was a pr merged to address the issue: #8644. The change since has been reverted with this pr: #9212

Would need a new pr to address the global application level lock issue
@saubyk saubyk added enhancement Improvements to existing features / behaviour kvdb database Related to the database/storage of LND postgres labels Oct 28, 2024
@Roasbeef Roasbeef added this to the v0.19.0 milestone Oct 28, 2024
@bhandras bhandras self-assigned this Oct 29, 2024
@aakselrod aakselrod linked a pull request Nov 2, 2024 that will close this issue
Open
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bhandras bhandras

Labels
database Related to the database/storage of LND enhancement Improvements to existing features / behaviour kvdb postgres
Projects
lnd v0.19
Status: In Progress
Milestone
v0.19.0
Development

Successfully merging a pull request may close this issue.

Reapply #8644 aakselrod/lnd
3 participants
@bhandras @Roasbeef @saubyk