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

fix: use insert ignore for filecache_extended #48564

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kesselb
Copy link
Contributor

@kesselb kesselb commented Oct 3, 2024

Summary

TODO

  • [ ]

Checklist

@kesselb kesselb added enhancement 2. developing Work in progress labels Oct 3, 2024
@kesselb kesselb added this to the Nextcloud 31 milestone Oct 3, 2024
@kesselb kesselb self-assigned this Oct 3, 2024
The current approach is to insert a new row and, if a unique constraint violation occurs, update an existing one. PostgreSQL logs the unique constraint violation as error "duplicate key value violates unique constraint".

Our Adapter.insertIgnoreConflict method provides a way to run an insert query without logging such errors by using the vendor-specific sql extensions like "on conflict do nothing" on Postgres.

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
@kesselb kesselb force-pushed the bug/19494/insert-ignore-conflict-for-filecache-extended branch from cda4948 to af89838 Compare October 7, 2024 11:30
@kesselb kesselb changed the title wip fix: use insert ignore for filecache_extended Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PostgreSQL duplicate key value violates unique constraint
1 participant