Merge pull request #13477 from backstage/freben/return

Update some comments now that knex has some partial sqlite3 returning() support
2022-09-01 12:00:58 +02:00
parent af7594d587 ac537f4302
commit 7e7263d3e2
2 changed files with 3 additions and 8 deletions
@@ -112,6 +112,7 @@ export class DefaultProcessingDatabase implements ProcessingDatabase {
    });

    // Delete old relations
+    // NOTE(freben): knex implemented support for returning() on update queries for sqlite, but at the current time of writing (Sep 2022) not for delete() queries.
    let previousRelationRows: DbRelationsRow[];
    if (
      tx.client.config.client.includes('sqlite3') ||