Skip to content

Feat: locking improvements with migration - #32

Open
gupta-apoorva wants to merge 21 commits into
karoo-ca:mainfrom
distribusion:feat/locking-improvements-with-migration
Open

gupta-apoorva wants to merge 21 commits into
karoo-ca:mainfrom
distribusion:feat/locking-improvements-with-migration

Conversation

@gupta-apoorva

@gupta-apoorva gupta-apoorva commented Aug 28, 2025 •

Copy link
Copy Markdown

This PR builds up on the work done by @oliverlambson in the locking-improvements PR by adding migrations.

@oliverlambson

Copy link
Copy Markdown
Contributor

@gupta-apoorva thanks for this - i made some light fixes to get linting and python<=3.10 tests passing.

I don't have time to properly review this right now, so apologies for the delay—will take a look on the weekend

@gupta-apoorva
gupta-apoorva force-pushed the feat/locking-improvements-with-migration branch from e205744 to f839348 Compare October 14, 2025 09:47
@gupta-apoorva
gupta-apoorva force-pushed the feat/locking-improvements-with-migration branch from f839348 to 47eed9c Compare October 14, 2025 09:48
@gupta-apoorva

Copy link
Copy Markdown
Author

@oliverlambson We found a limitation with pg_notify payload which I have fixed by updating the NOTIFY_EXISTING_MESSAGES_QUERY. Do have a look and let me know if I can help in any other way.

@hmax

hmax commented Dec 22, 2025

Copy link
Copy Markdown

Hey, any news on this one? Would love to see it upstreamed

@hmax

hmax commented Mar 20, 2026

Copy link
Copy Markdown

I've actually encountered a problem with the limit and found out that there's an assumption of 4 bytes per int, but we convert it to ::text, which cuts our budget significantly, if we consider large ids, down to ~700 if we use serial/integer and ~360 if there's a switch to bigserial

@hmax
hmax force-pushed the feat/locking-improvements-with-migration branch from d7155f6 to 23052c4 Compare August 7, 2026 14:24
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.

3 participants