fix review

schema-engine/connectors/mongodb-schema-connector/src/sampler.rs

@@ -11,6 +11,7 @@ use mongodb::{
 use mongodb_schema_describer::MongoSchema;
 use schema_connector::{warnings::Model, IntrospectionContext, IntrospectionResult, Warnings};
 use statistics::*;
+use std::borrow::Cow;
 
 /// From the given database, lists all collections as models, and samples
 /// maximum of SAMPLE_SIZE documents for their fields with the following rules:
@@ -66,7 +67,7 @@ pub(super) async fn sample(
     for file_id in ctx.previous_schema().db.iter_file_ids() {
         let file_name = ctx.previous_schema().db.file_name(file_id);
 
-        data_model.create_empty_file(file_name.to_string());
+        data_model.create_empty_file(Cow::Borrowed(file_name));
     }
 
     statistics.render(ctx, &mut data_model, &mut warnings);

schema-engine/connectors/mongodb-schema-connector/src/sampler/statistics.rs

@@ -20,6 +20,7 @@ use once_cell::sync::Lazy;
 use psl::datamodel_connector::constraint_names::ConstraintNames;
 use regex::Regex;
 use std::{
+    borrow::Cow,
     cmp::Ordering,
     collections::{BTreeMap, HashMap, HashSet},
     fmt,
@@ -416,7 +417,7 @@ impl<'a> Statistics<'a> {
                 None => ctx.introspection_file_name(),
             };
 
-            rendered.push_composite_type(file_name.to_string(), r#type);
+            rendered.push_composite_type(Cow::Borrowed(file_name), r#type);
         }
 
         for (model_name, model) in models.into_iter() {
@@ -425,7 +426,7 @@ impl<'a> Statistics<'a> {
                 None => ctx.introspection_file_name(),
             };
 
-            rendered.push_model(file_name.to_string(), model);
+            rendered.push_model(Cow::Borrowed(file_name), model);
         }
     }
 

schema-engine/connectors/sql-schema-connector/src/introspection/rendering.rs

@@ -15,6 +15,7 @@ use crate::introspection::datamodel_calculator::DatamodelCalculatorContext;
 use datamodel_renderer as renderer;
 use psl::PreviewFeature;
 use schema_connector::ViewDefinition;
+use std::borrow::Cow;
 
 /// Combines the SQL database schema and an existing PSL schema to a
 /// PSL schema definition string.
@@ -29,7 +30,7 @@ pub(crate) fn to_psl_string(
     for file_id in ctx.previous_schema.db.iter_file_ids() {
         let file_name = ctx.previous_schema.db.file_name(file_id);
 
-        datamodel.create_empty_file(file_name.to_string());
+        datamodel.create_empty_file(Cow::Borrowed(file_name));
     }
 
     enums::render(introspection_file_name, ctx, &mut datamodel);

schema-engine/connectors/sql-schema-connector/src/introspection/rendering/views.rs

@@ -4,10 +4,11 @@ use crate::introspection::{
 };
 use datamodel_renderer::datamodel as renderer;
 use schema_connector::ViewDefinition;
+use std::borrow::Cow;
 
 /// Render all view blocks to the PSL.
 pub(super) fn render<'a>(
-    introspection_file_name: &str,
+    introspection_file_name: &'a str,
     ctx: &'a DatamodelCalculatorContext<'a>,
     rendered: &mut renderer::Datamodel<'a>,
 ) -> Vec<ViewDefinition> {
@@ -39,7 +40,7 @@ pub(super) fn render<'a>(
             None => introspection_file_name,
         };
 
-        rendered.push_view(file_name.to_string(), render);
+        rendered.push_view(Cow::Borrowed(file_name), render);
     }
 
     definitions