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

Updates should lock around the location they are updating #67

Open
Mr0grog opened this issue Apr 20, 2021 · 3 comments
Open

Updates should lock around the location they are updating #67

Mr0grog opened this issue Apr 20, 2021 · 3 comments

Comments

@Mr0grog
Copy link
Collaborator

Mr0grog commented Apr 20, 2021

When receiving data a high-frequency, we are sometimes running into issues where two parallel calls update the same location and both either try to create the same new update record or the same new location. We should probably use PG advisory locks on the ID or external IDs to prevent those calls from colliding.

@Mr0grog
Copy link
Collaborator Author

Mr0grog commented Jan 13, 2022

We probably need to do #463 before or as part of this.

@Mr0grog
Copy link
Collaborator Author

Mr0grog commented Jun 19, 2023

Additional cleanup since issue was originally written has largely mitigated the issue, but it would still be good to do this correctly as suggested here.

@Mr0grog
Copy link
Collaborator Author

Mr0grog commented Jun 19, 2023

This project has been shut down. This issue has been left open as a guide for anyone forking this project — addressing this issue (or at least knowing about it!) is likely to be worthwhile for you if you are maintaining a running copy or fork of UNIVAF.

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