Skip to content

🐛(backend) move lock table closer to the insert operation targeted#2132

Merged
lunika merged 3 commits intomainfrom
changed/move-lock-tables
Mar 25, 2026
Merged

🐛(backend) move lock table closer to the insert operation targeted#2132
lunika merged 3 commits intomainfrom
changed/move-lock-tables

Conversation

@lunika
Copy link
Member

@lunika lunika commented Mar 25, 2026

Purpose

We want to lock the table just before the insert we want to protect is made. In the case of the perform_create action in the Document viewset, an http call is made after the lock and can take a very long time, blocking for nothing the table.

Proposal

  • 🐛(backend) move lock table closer to the insert operation targeted

We want to lock the table just before the insert we want to protect is
made. In the case of the perform_create action in the Document viewset,
an http call is made after the lock and can take a very long time,
blocking for nothing the table.
@lunika lunika requested review from jmaupetit and qbey March 25, 2026 14:46
@lunika lunika self-assigned this Mar 25, 2026
Copy link

@jmaupetit jmaupetit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch!

lunika and others added 2 commits March 25, 2026 17:22
We added logs on the conversion made when a doc is created.
By default, Node.js has a memory limit of
around 512MB, which can lead to out-of-memory
errors when processing large documents.
This commit increases the memory limit to
2GB for the y-provider server, allowing
it to handle larger documents without crashing.
@lunika lunika force-pushed the changed/move-lock-tables branch from 930bcfd to cbe6a67 Compare March 25, 2026 16:22
@lunika lunika merged commit cbe6a67 into main Mar 25, 2026
33 of 34 checks passed
@lunika lunika deleted the changed/move-lock-tables branch March 25, 2026 16:35
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