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

Improvement: Move the last written LSN (LwLSN) cache into neon extension #10973

Open
ololobus opened this issue Feb 25, 2025 · 1 comment · May be fixed by #11031 or neondatabase/postgres#590
Open

Improvement: Move the last written LSN (LwLSN) cache into neon extension #10973

ololobus opened this issue Feb 25, 2025 · 1 comment · May be fixed by #11031 or neondatabase/postgres#590
Assignees
Labels
c/compute Component: compute, excluding postgres itself

Comments

@ololobus
Copy link
Member

ololobus commented Feb 25, 2025

Problem

It's currently maintained in our Postgres forks, and we are approaching 5 of them (PG 14-18) this year, so more code creates an additional maintenance overhead.

Proposal

Move this code to the extension and replace the Postgres part with hooks.

We had a very old attempt to approach it: #3193. It probably doesn't make much sense to try to revive it and rebase, as there have been too many changes since then, but it might be good to grasp the idea.

@thesuhas
Copy link
Contributor

Added first draft code in: #11031

Started removing from postgres v17 in: neondatabase/postgres#590

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c/compute Component: compute, excluding postgres itself
Projects
None yet
2 participants