Fix: Backfilling batch size for seeds should always be None (#979)

Author: izeigerman

sqlmesh/core/model/definition.py

@@ -872,6 +872,13 @@ def seed_path(self) -> Path:
     def depends_on_past(self) -> bool:
         return False
 
+    @property
+    def batch_size(self) -> t.Optional[int]:
+        # Unlike other model kinds, the batch size provided in the SEED kind represents the
+        # maximum number of rows to insert in a single batch.
+        # We should never batch intervals for seed models.
+        return None
+
     def to_dehydrated(self) -> SeedModel:
         """Creates a dehydrated copy of this model.
 

tests/core/test_model.py

@@ -456,6 +456,8 @@ def test_seed():
         "name": exp.DataType.build("varchar"),
     }
 
+    assert model.batch_size is None
+
 
 def test_seed_provided_columns():
     expressions = parse(

tests/core/test_snapshot.py

@@ -432,7 +432,7 @@ def test_fingerprint_seed_model():
 
     expected_fingerprint = SnapshotFingerprint(
         data_hash="3834815287",
-        metadata_hash="1624292983",
+        metadata_hash="1120323454",
     )
 
     model = load_model(expressions, path=Path("./examples/sushi/models/test_model.sql"))