fix: Only initialize shape and scratch-shape if this is a brand n…

…ew Combiner, otherwise keep the schema around to reduce spurious logging
estuary · Aug 4, 2023 · 8601c59 · 8601c59
1 parent 35e3677
commit 8601c59
Showing 1 changed file with 8 additions and 8 deletions.
diff --git a/crates/derive/src/combine_api.rs b/crates/derive/src/combine_api.rs
@@ -133,14 +133,14 @@ impl cgo::Service for API {
 
                 self.state = Some(State {
                     combiner,
-                    shape: Shape {
-                        type_: types::INVALID,
-                        ..Default::default()
-                    },
-                    scratch_shape: Shape {
-                        type_: types::INVALID,
-                        ..Default::default()
-                    },
+                    shape: self
+                        .state
+                        .as_ref()
+                        .map_or(Shape::invalid(), |state| state.shape.clone()),
+                    scratch_shape: self
+                        .state
+                        .as_ref()
+                        .map_or(Shape::invalid(), |state| state.scratch_shape.clone()),
                     fields_ex,
                     key_ex,
                     uuid_placeholder_ptr,