Make Upsert distinguish between Unique key constraint and foreign key constraint [243039555]

Fixed

Bug

Status Update

No update yet.

Description

pi...@google.com

created issue #1

Aug 18, 2022 06:33PM

In EntityUpsertionAdapter add if-statement to check error message catch. When the message is FK violation, throw a new FK violation error. When the message is UK violation, update the database.

Comments

ap...@google.com <ap...@google.com> #2Aug 26, 2022 03:32PM

The same happens when trying to install eap versions of the plugin (e.g. 1.1.3-eap-85-Studio3.0-1).

is...@google.com <is...@google.com> Sep 20, 2022 08:58AM

Status: New

da...@google.com <da...@google.com> Sep 20, 2022 12:39PM

Marked as fixed, reassigned to da...@google.com.

na...@google.com <na...@google.com> #3Oct 5, 2022 05:14PM

Yes, we had to temporarily disable updates. This will be fixed in Canary 5.

pe...@gmail.com <pe...@gmail.com> #4Jan 5, 2023 05:03PM

Ah, that explains. Thanks!

da...@google.com <da...@google.com> #5Jan 18, 2023 02:55PM

Status: Assigned (reopened)

rasgdfzsxgdzfxc

pe...@gmail.com <pe...@gmail.com> #6Jan 20, 2023 07:01PM

mi...@gmail.com <mi...@gmail.com> #7Jul 31, 2023 08:03AM

Hello,

I've just hit this issue and I would like to know what's the status on handling 2067 SQLITE_CONSTRAINT_UNIQUE the same way and proceed to an update. I was very surprised upsert wasn't working.

Thanks

el...@google.com <el...@google.com> #8Jul 31, 2023 12:21PM

Thanks for the bump, we'll resolve this issue asap.

ap...@google.com <ap...@google.com> #9Aug 7, 2023 08:47PM

Project: platform/frameworks/support
Branch: androidx-main

commit 01002bedc549e0264a5f920dc554ed7dd7785cfe
Author: elifbilgin <elifbilgin@google.com>
Date: Tue Aug 01 05:53:50 2023

Handle special case SQLite exception during upsert.

When 2067 SQLITE_CONSTRAINT_UNIQUE is thrown during an upsert, upsert should perform an update.

Bug: 243039555
Test: EntityUpsertionAdapterTest.java
Change-Id: If28499c59443f590ac456924eff03b18f1a87e4f

M room/integration-tests/testapp/src/androidTest/java/androidx/room/integration/testapp/test/EntityUpsertionAdapterTest.java
M room/room-runtime/src/main/java/androidx/room/EntityUpsertionAdapter.kt

https://android-review.googlesource.com/2686153

el...@google.com <el...@google.com> Aug 7, 2023 09:51PM

Marked as fixed.