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

Concurrency Issues with doc.save #6

Open
uncleJim21 opened this issue Oct 9, 2023 · 0 comments
Open

Concurrency Issues with doc.save #6

uncleJim21 opened this issue Oct 9, 2023 · 0 comments

Comments

@uncleJim21
Copy link
Contributor

await doc.save();

@CoachChuckFF I noticed an issue with this when trying to spin up a new service.

It seems that the way you're doing it can actually cause the same doc to try to get saved more than once and crash the server. Likewise you are also assuming that the user will continually poll the endpoint bc you seem to give them an out of date state. I get that it works to first order but may be a good idea to refactor this.

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

No branches or pull requests

1 participant