Use upsert when inserting read receipts (matrix-org#7607)

Fixes matrix-org#7469

Signed-off-by: Dagfinn Ilmari Mannsåker <ilmari@ilmari.org>

Author: ilmari

changelog.d/7607.bugfix

@@ -0,0 +1 @@
+Fix duplicate key violation when persisting read markers.

synapse/storage/data_stores/main/receipts.py

@@ -391,27 +391,22 @@ def insert_linearized_receipt_txn(
             (user_id, room_id, receipt_type),
         )
 
-        self.db.simple_delete_txn(
+        self.db.simple_upsert_txn(
             txn,
             table="receipts_linearized",
             keyvalues={
                 "room_id": room_id,
                 "receipt_type": receipt_type,
                 "user_id": user_id,
             },
-        )
-
-        self.db.simple_insert_txn(
-            txn,
-            table="receipts_linearized",
             values={
                 "stream_id": stream_id,
-                "room_id": room_id,
-                "receipt_type": receipt_type,
-                "user_id": user_id,
                 "event_id": event_id,
                 "data": json.dumps(data),
             },
+            # receipts_linearized has a unique constraint on
+            # (user_id, room_id, receipt_type), so no need to lock
+            lock=False,
         )
 
         if receipt_type == "m.read" and stream_ordering is not None: