fix(db-*): do not exit process on error during connect (#14647)

This PR removes `process.exit(1)` from our exception handlers within db
connect methods. When a connection error occurred in vercel fluid
compute, it ungracefully shut down the entire compute process.

By re-throwing the error instead, [vercel is able to gracefully handle
the
error](https://vercel.com/changelog/improved-unhandled-node-js-errors-in-fluid-compute)
without shutting down concurrent requests running on the same compute
instance.

process.exit(1) [was initially added by this
commit](f3fee34#r169957812),
however noone was able to recall why it was added.

Author: AlessioGr

packages/db-d1-sqlite/src/connect.ts

@@ -51,7 +51,7 @@ export const connect: Connect = async function connect(
       this.rejectInitializing()
     }
     console.error(err)
-    process.exit(1)
+    throw new Error(`Error: cannot connect to SQLite: ${message}`)
   }
 
   // Only push schema if not in production

packages/db-mongodb/src/connect.ts

@@ -108,6 +108,6 @@ export const connect: Connect = async function connect(
       err,
       msg,
     })
-    process.exit(1)
+    throw new Error(`Error: cannot connect to MongoDB: ${msg}`)
   }
 }

packages/db-postgres/src/connect.ts

@@ -108,7 +108,7 @@ export const connect: Connect = async function connect(
     if (typeof this.rejectInitializing === 'function') {
       this.rejectInitializing()
     }
-    process.exit(1)
+    throw new Error(`Error: cannot connect to Postgres: ${err.message}`)
   }
 
   await this.createExtensions()

packages/db-sqlite/src/connect.ts

@@ -36,7 +36,7 @@ export const connect: Connect = async function connect(
     if (typeof this.rejectInitializing === 'function') {
       this.rejectInitializing()
     }
-    process.exit(1)
+    throw new Error(`Error: cannot connect to SQLite: ${message}`)
   }
 
   // Only push schema if not in production

packages/db-vercel-postgres/src/connect.ts

@@ -90,7 +90,7 @@ export const connect: Connect = async function connect(
     if (typeof this.rejectInitializing === 'function') {
       this.rejectInitializing()
     }
-    process.exit(1)
+    throw new Error(`Error: cannot connect to Postgres: ${err.message}`)
   }
 
   await this.createExtensions()

packages/drizzle/src/transactions/beginTransaction.ts

@@ -59,7 +59,7 @@ export const beginTransaction: BeginTransaction = async function beginTransactio
     }
   } catch (err) {
     this.payload.logger.error({ err, msg: `Error: cannot begin transaction: ${err.message}` })
-    process.exit(1)
+    throw new Error(`Error: cannot begin transaction: ${err.message}`)
   }
 
   return id

packages/payload/src/index.ts

@@ -1133,12 +1133,12 @@ export const getPayload = async (
     return cached.payload
   }
 
-  if (!cached.promise) {
-    // no need to await options.config here, as it's already awaited in the BasePayload.init
-    cached.promise = new BasePayload().init(options)
-  }
-
   try {
+    if (!cached.promise) {
+      // no need to await options.config here, as it's already awaited in the BasePayload.init
+      cached.promise = new BasePayload().init(options)
+    }
+
     cached.payload = await cached.promise
 
     if (