From bb1e5c755e763306325b4cb48f15260563722df9 Mon Sep 17 00:00:00 2001 From: Alexey Orlenko Date: Mon, 11 Aug 2025 16:57:08 +0200 Subject: [PATCH] don't ignore errors in migrate [integration] --- .../connectors/sql-schema-connector/src/flavour.rs | 12 +----------- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/schema-engine/connectors/sql-schema-connector/src/flavour.rs b/schema-engine/connectors/sql-schema-connector/src/flavour.rs index b800f5c5089..b36d58b66b3 100644 --- a/schema-engine/connectors/sql-schema-connector/src/flavour.rs +++ b/schema-engine/connectors/sql-schema-connector/src/flavour.rs @@ -248,17 +248,7 @@ pub(crate) trait SqlConnector: Send + Sync + Debug { { return Ok(Err(PersistenceNotInitializedError)); } - Err(_) => { - // TODO: this is a workaround, as currently the errors thrown by Driver Adapters do not - // match the known user-facing errors we expect. - // We should fix this in the future. - // - // This used to actually yield: - // ``` - // err @ Err(_) => err? - // ``` - return Ok(Err(PersistenceNotInitializedError)); - } + Err(err) => return Err(err), }; let rows = rows