Better logging of migrations

mozilla-services · Mar 5, 2024 · c8c1458 · c8c1458
1 parent e2b8563
commit c8c1458
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/tokenserver-db-mysql/src/pool.rs b/tokenserver-db-mysql/src/pool.rs
@@ -13,8 +13,11 @@ embed_migrations!();
 /// begin_test_transaction during tests. So this runs on its own separate conn.
 pub fn run_embedded_migrations(database_url: &str) -> DbResult<()> {
     let conn = MysqlConnection::establish(database_url)?;
-
+    #[cfg(debug_assertions)]
+    // XXX: this doesn't show the DDL statements
+    // https://github.com/shssoichiro/diesel-logger/issues/1
     embedded_migrations::run(&LoggingConnection::new(conn))?;
-
+    #[cfg(not(debug_assertions))]
+    embedded_migrations::run(&conn)?;
     Ok(())
 }
diff --git a/tokenserver-db-sqlite/src/pool.rs b/tokenserver-db-sqlite/src/pool.rs
@@ -11,7 +11,12 @@ embed_migrations!();
 pub fn run_embedded_migrations(database_url: &str) -> DbResult<()> {
     let conn = SqliteConnection::establish(database_url)?;
 
+    #[cfg(debug_assertions)]
+    // XXX: this doesn't show the DDL statements
+    // https://github.com/shssoichiro/diesel-logger/issues/1
     embedded_migrations::run(&LoggingConnection::new(conn))?;
+    #[cfg(not(debug_assertions))]
+    embedded_migrations::run(&conn)?;
 
     Ok(())
 }