fix: Drop foreign key constraint on order id

The order and the matches are persisted by async actors. It can happen that the market order has not yet been persisted when the match is processed.

Thus it could happen that the matches table is persisted before the order has been persisted, which resulted in a foreign key violation. By dropping the foreign key we don't care if the order is already persisted and can continue with eventual consistency.

I think this is the better option than to synchronize the two inserts as we are anyways aiming for decoupling the orderbook from the coordinator.

coordinator/migrations/2024-06-07-105918_drop_foreign_key_constraint_to_order_id_on_matches/down.sql

@@ -0,0 +1,9 @@
+ALTER TABLE "matches"
+    ADD CONSTRAINT matches_order_id_fkey
+        FOREIGN KEY (order_id)
+            REFERENCES orders (order_id);
+
+ALTER TABLE "matches"
+    ADD CONSTRAINT matches_match_order_id_fkey
+        FOREIGN KEY (match_order_id)
+            REFERENCES orders (order_id);

coordinator/migrations/2024-06-07-105918_drop_foreign_key_constraint_to_order_id_on_matches/up.sql

@@ -0,0 +1,2 @@
+ALTER TABLE "matches" DROP CONSTRAINT matches_order_id_fkey;
+ALTER TABLE "matches" DROP CONSTRAINT matches_match_order_id_fkey;

coordinator/src/schema.rs

@@ -216,23 +216,6 @@ diesel::table! {
     }
 }
 
-diesel::table! {
-    use diesel::sql_types::*;
-    use super::sql_types::InvoiceStateType;
-
-    hodl_invoices (id) {
-        id -> Int4,
-        trader_pubkey -> Text,
-        r_hash -> Text,
-        amount_sats -> Int8,
-        pre_image -> Nullable<Text>,
-        created_at -> Timestamptz,
-        updated_at -> Nullable<Timestamptz>,
-        invoice_state -> InvoiceStateType,
-        order_id -> Nullable<Uuid>,
-    }
-}
-
 diesel::table! {
     funding_fee_events (id) {
         id -> Int4,
@@ -257,6 +240,23 @@ diesel::table! {
     }
 }
 
+diesel::table! {
+    use diesel::sql_types::*;
+    use super::sql_types::InvoiceStateType;
+
+    hodl_invoices (id) {
+        id -> Int4,
+        trader_pubkey -> Text,
+        r_hash -> Text,
+        amount_sats -> Int8,
+        pre_image -> Nullable<Text>,
+        created_at -> Timestamptz,
+        updated_at -> Nullable<Timestamptz>,
+        invoice_state -> InvoiceStateType,
+        order_id -> Nullable<Uuid>,
+    }
+}
+
 diesel::table! {
     last_outbound_dlc_messages (peer_id) {
         peer_id -> Text,